New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PARQUET-3: tool to merge pull requests based on Spark #5

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@julienledem
Member

julienledem commented Jun 19, 2014

given a pull request id on github.com/apache/incubator-parquet-mr this script will merge it
requires 2 remotes: apache-github and apache to point to the corresponding repos.
tested here (pretending my fork is the apache remote):
julienledem@485658a

original tool:
https://github.com/apache/spark/blob/master/dev/merge_spark_pr.py

@julienledem julienledem changed the title from tool to merge pull requests based on Spark to PARQUET-3: tool to merge pull requests based on Spark Jun 19, 2014

@dvryaboy

This comment has been minimized.

Show comment
Hide comment
@dvryaboy

dvryaboy Jun 19, 2014

Contributor

Nice. We should add a .md doc that includes instruction on this, and reference it from the "how to contribute" section of the readme.

Contributor

dvryaboy commented Jun 19, 2014

Nice. We should add a .md doc that includes instruction on this, and reference it from the "how to contribute" section of the readme.

@julienledem

This comment has been minimized.

Show comment
Hide comment
@julienledem

julienledem Jun 20, 2014

Member

Added README on how to use it.

Member

julienledem commented Jun 20, 2014

Added README on how to use it.

@julienledem

This comment has been minimized.

Show comment
Hide comment
@julienledem
Member

julienledem commented Jun 20, 2014

@asfgit asfgit closed this in 859b6b4 Jun 21, 2014

@julienledem julienledem deleted the julienledem:merge_pr_tool branch Jun 23, 2014

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