Skip to content

Conversation

@yhuai
Copy link
Contributor

@yhuai yhuai commented Jan 18, 2017

What changes were proposed in this pull request?

Update known_translations per #16423 (comment)

@yhuai
Copy link
Contributor Author

yhuai commented Jan 18, 2017

cc @lw-lin

@lw-lin
Copy link
Contributor

lw-lin commented Jan 18, 2017

this lgtm, thanks!

@SparkQA
Copy link

SparkQA commented Jan 18, 2017

Test build #71546 has finished for PR 16628 at commit bf2f532.

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

@lw-lin
Copy link
Contributor

lw-lin commented Jan 18, 2017

sorry, I made a mistake. It should really be Yuhai Cen, rather than Yuhai Ceng. could you fix it?

@yhuai
Copy link
Contributor Author

yhuai commented Jan 18, 2017

done

@SparkQA
Copy link

SparkQA commented Jan 19, 2017

Test build #71621 has finished for PR 16628 at commit 7cb484f.

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

@yhuai
Copy link
Contributor Author

yhuai commented Jan 19, 2017

I am merging this to master.

@asfgit asfgit closed this in 0c92318 Jan 19, 2017
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
## What changes were proposed in this pull request?
Update known_translations per apache#16423 (comment)

Author: Yin Huai <[email protected]>

Closes apache#16628 from yhuai/known_translations.
cmonkey pushed a commit to cmonkey/spark that referenced this pull request Feb 15, 2017
## What changes were proposed in this pull request?
Update known_translations per apache#16423 (comment)

Author: Yin Huai <[email protected]>

Closes apache#16628 from yhuai/known_translations.
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