Skip to content

Conversation

@HyukjinKwon
Copy link
Member

@HyukjinKwon HyukjinKwon commented Jul 30, 2017

What changes were proposed in this pull request?

This PR proposes to fix few rather typos in merge_spark_pr.py.

  • # usage: ./apache-pr-merge.py (see config env vars below)
    -> # usage: ./merge_spark_pr.py (see config env vars below)

  • ... have local a Spark ... -> ... have a local Spark ...

  • ... to Apache. -> ... to Apache Spark.

I skimmed this file and these look all I could find.

How was this patch tested?

pep8 check (./dev/lint-python).

@SparkQA
Copy link

SparkQA commented Jul 30, 2017

Test build #80054 has finished for PR 18776 at commit b511b6d.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@HyukjinKwon
Copy link
Member Author

I guess it is safe to merge. Thanks for approval @srowen.

@HyukjinKwon
Copy link
Member Author

Merged to master.

@asfgit asfgit closed this in f1a798b Jul 31, 2017
@HyukjinKwon HyukjinKwon deleted the minor-merge-script branch January 2, 2018 03:41
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