Skip to content
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

[SPARK-18754][SS] Rename recentProgresses to recentProgress #16182

Closed
wants to merge 2 commits into from

Conversation

marmbrus
Copy link
Contributor

@marmbrus marmbrus commented Dec 7, 2016

Based on an informal survey, users find this option easier to understand / remember.

@marmbrus
Copy link
Contributor Author

marmbrus commented Dec 7, 2016

/cc @tdas

@tdas
Copy link
Contributor

tdas commented Dec 7, 2016

LGTM.

@SparkQA
Copy link

SparkQA commented Dec 7, 2016

Test build #69761 has finished for PR 16182 at commit 184a6d1.

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

@tdas
Copy link
Contributor

tdas commented Dec 7, 2016

retest this please.

@SparkQA
Copy link

SparkQA commented Dec 7, 2016

Test build #69771 has finished for PR 16182 at commit 184a6d1.

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

@tdas
Copy link
Contributor

tdas commented Dec 7, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Dec 7, 2016

Test build #69774 has started for PR 16182 at commit 184a6d1.

@SparkQA
Copy link

SparkQA commented Dec 7, 2016

Test build #3471 has started for PR 16182 at commit 184a6d1.

@SparkQA
Copy link

SparkQA commented Dec 7, 2016

Test build #3472 has started for PR 16182 at commit 184a6d1.

@zsxwing
Copy link
Member

zsxwing commented Dec 7, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Dec 7, 2016

Test build #69811 has finished for PR 16182 at commit 184a6d1.

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

@tdas
Copy link
Contributor

tdas commented Dec 7, 2016

Merging this to master and 2.1

@asfgit asfgit closed this in 70b2bf7 Dec 7, 2016
asfgit pushed a commit that referenced this pull request Dec 7, 2016
Based on an informal survey, users find this option easier to understand / remember.

Author: Michael Armbrust <[email protected]>

Closes #16182 from marmbrus/renameRecentProgress.

(cherry picked from commit 70b2bf7)
Signed-off-by: Tathagata Das <[email protected]>
robert3005 pushed a commit to palantir/spark that referenced this pull request Dec 15, 2016
Based on an informal survey, users find this option easier to understand / remember.

Author: Michael Armbrust <[email protected]>

Closes apache#16182 from marmbrus/renameRecentProgress.
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
Based on an informal survey, users find this option easier to understand / remember.

Author: Michael Armbrust <[email protected]>

Closes apache#16182 from marmbrus/renameRecentProgress.
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.

4 participants