Github user squito commented on the issue:
https://github.com/apache/spark/pull/20082
sorry this is a little late, but lgtm too. agree with the points above
about leaving the old name deprecated and moving to the new name
---
Github user advancedxy commented on the issue:
https://github.com/apache/spark/pull/20082
Of course.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/20082
@advancedxy could you also submit a PR to add a `def stageAttemptNumber =
stageAttemptId` to `StageInfo` and mark `stageAttemptId` as deprecated? It will
make our public APIs more consistent.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20082
thanks, merging to master/2.3!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20082
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user advancedxy commented on the issue:
https://github.com/apache/spark/pull/20082
@cloud-fan Please take another look.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85513/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85513 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85513/testReport)**
for PR 20082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85512/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85512 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85512/testReport)**
for PR 20082 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85513 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85513/testReport)**
for PR 20082 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85512 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85512/testReport)**
for PR 20082 at commit
Github user advancedxy commented on the issue:
https://github.com/apache/spark/pull/20082
ping @cloud-fan @jiangxb1987 @zsxwing, I think it's ready for merging.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85474/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85474 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85474/testReport)**
for PR 20082 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85474 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85474/testReport)**
for PR 20082 at commit
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20082
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85457/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85457 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85457/testReport)**
for PR 20082 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85457 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85457/testReport)**
for PR 20082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85429/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85429 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85429/testReport)**
for PR 20082 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85429 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85429/testReport)**
for PR 20082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85421/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85421 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85421/testReport)**
for PR 20082 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85421 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85421/testReport)**
for PR 20082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85420 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85420/testReport)**
for PR 20082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85420/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85420 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85420/testReport)**
for PR 20082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85410/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85410 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85410/testReport)**
for PR 20082 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85410 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85410/testReport)**
for PR 20082 at commit
Github user advancedxy commented on the issue:
https://github.com/apache/spark/pull/20082
>Please explain in the JIRA/PR description about your use case that need to
access the stageAttemptId in the executor - we must provide a very good reason
to make such a change to a public
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85409 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85409/testReport)**
for PR 20082 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85409/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20082
**[Test build #85409 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85409/testReport)**
for PR 20082 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20082
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20082
Please explain in the JIRA/PR description about your use case that need to
access the stageAttemptId in the executor - we must provide a very good reason
to make such a change to a public
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20082
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user advancedxy commented on the issue:
https://github.com/apache/spark/pull/20082
cc @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
49 matches
Mail list logo