Skip to content

Conversation

@cloud-fan
Copy link
Contributor

What changes were proposed in this pull request?

update known_translations after running translate-contributors.py during 2.4.0 release

How was this patch tested?

N/A

@cloud-fan
Copy link
Contributor Author

cc @gatorsmile

@SparkQA
Copy link

SparkQA commented Nov 5, 2018

Test build #98478 has finished for PR 22949 at commit 6cd3737.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyun
Copy link
Member

Retest this please.

e-dorigatti - Emilio Dorigatti
eric-maynard - Eric Maynard
felixalbani - Felix Albani
fjh100456 - fjh100456
Copy link
Member

Choose a reason for hiding this comment

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

Jinhua Fu

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah I missed this one.

huangweizhe123 - Weizhe Huang
ivoson - Tengfei Huang
jinxing64 - Jin Xing
liu-zhaokun - Zhaokun Liu
Copy link
Member

Choose a reason for hiding this comment

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

Also add

KaiXinXiaoLei - LeiLei Hu

@SparkQA
Copy link

SparkQA commented Nov 5, 2018

Test build #98485 has finished for PR 22949 at commit 6cd3737.

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

@wangyum
Copy link
Member

wangyum commented Nov 6, 2018

Update line 132:

wangyum - Yuming Wang

How about add:

lipzhu - Lipeng Zhu
daviddingly - David Ding
laskfla - Keith Sun

@cloud-fan
Copy link
Contributor Author

Note that, these updates are generated by the script not me. If someone is not in the list, it means the script can figure out the full name without translation.

@SparkQA
Copy link

SparkQA commented Nov 6, 2018

Test build #98498 has finished for PR 22949 at commit 9808e43.

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

@gatorsmile
Copy link
Member

gatorsmile commented Nov 6, 2018

Thanks! Merged to master and 2.4.

asfgit pushed a commit that referenced this pull request Nov 6, 2018
## What changes were proposed in this pull request?

update known_translations after running `translate-contributors.py` during 2.4.0 release

## How was this patch tested?

N/A

Closes #22949 from cloud-fan/contributors.

Authored-by: Wenchen Fan <[email protected]>
Signed-off-by: gatorsmile <[email protected]>
(cherry picked from commit a241a15)
Signed-off-by: gatorsmile <[email protected]>
@asfgit asfgit closed this in a241a15 Nov 6, 2018
jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
## What changes were proposed in this pull request?

update known_translations after running `translate-contributors.py` during 2.4.0 release

## How was this patch tested?

N/A

Closes apache#22949 from cloud-fan/contributors.

Authored-by: Wenchen Fan <[email protected]>
Signed-off-by: gatorsmile <[email protected]>
kai-chi pushed a commit to kai-chi/spark that referenced this pull request Jul 23, 2019
## What changes were proposed in this pull request?

update known_translations after running `translate-contributors.py` during 2.4.0 release

## How was this patch tested?

N/A

Closes apache#22949 from cloud-fan/contributors.

Authored-by: Wenchen Fan <[email protected]>
Signed-off-by: gatorsmile <[email protected]>
(cherry picked from commit a241a15)
Signed-off-by: gatorsmile <[email protected]>
kai-chi pushed a commit to kai-chi/spark that referenced this pull request Aug 1, 2019
## What changes were proposed in this pull request?

update known_translations after running `translate-contributors.py` during 2.4.0 release

## How was this patch tested?

N/A

Closes apache#22949 from cloud-fan/contributors.

Authored-by: Wenchen Fan <[email protected]>
Signed-off-by: gatorsmile <[email protected]>
(cherry picked from commit a241a15)
Signed-off-by: gatorsmile <[email protected]>
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.

7 participants