Skip to content

PARQUET-667: Update committers lists to point to apache website - #355

Closed
isnotinvain wants to merge 1 commit into
apache:masterfrom
isnotinvain:alexlevenson/committers-update
Closed

isnotinvain wants to merge 1 commit into
apache:masterfrom
isnotinvain:alexlevenson/committers-update

Conversation

@isnotinvain

Copy link
Copy Markdown
Contributor

We have two out of date lists of committers in .md files in the repo.
This PR changes them to point to the apache website which should stay up to date automatically.

@isnotinvain

Copy link
Copy Markdown
Contributor Author

@julienledem / @rdblue does this look OK?

@rdblue

rdblue commented Jul 27, 2016

Copy link
Copy Markdown
Contributor

+1

@isnotinvain

Copy link
Copy Markdown
Contributor Author

Thanks!

@isnotinvain isnotinvain changed the title Update committers lists to point to apache website PARQUET-667: Update committers lists to point to apache website Jul 27, 2016
@asfgit asfgit closed this in 60b6d5a Jul 27, 2016
Comment thread dev/COMMITTERS.md
| Tianshuo Deng | tianshuo | tsdeng | |
| Tom White | tomwhite | tomwhite | |
| Wesley Graham Peck | wesleypeck | wesleypeck | |
Please see: [Apache Parquet Committers and PMC](http://people.apache.org/committers-by-project.html#parquet)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the apache website does not contain the corresponding github id and jira id. The goal of this doc was to help contributors to ping committers on github/jira.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see. This doc has become out of date though (and will likely drift again). Any suggestions?
We could just manually try to keep it in sync.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

by out of date, really we mean that there are missing names to the list. The data present was valid.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, that's what I meant.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@julienledem added back + updated in #360

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants