You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2020/02/14 07:44:06 UTC
[GitHub] [spark] HeartSaVioR opened a new pull request #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
HeartSaVioR opened a new pull request #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576
### What changes were proposed in this pull request?
This patch addresses the post-hoc review comment linked here - https://github.com/apache/spark/pull/25670#discussion_r373304076
### Why are the changes needed?
We would like to explicitly document the direct relationship before we finish up structuring of configurations.
### Does this PR introduce any user-facing change?
No.
### How was this patch tested?
N/A
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586286401
Merged build finished. Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586188012
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23173/
Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586596543
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118474/
Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586155596
Merged build finished. Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586284578
**[Test build #118415 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118415/testReport)** for PR 27576 at commit [`5760af9`](https://github.com/apache/spark/commit/5760af975db77aaeadd1eef6f552f64bacb09ba1).
* This patch **fails PySpark unit tests**.
* This patch merges cleanly.
* This patch adds no public classes.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586142517
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23156/
Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586143086
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118399/
Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586188012
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23173/
Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HeartSaVioR commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
HeartSaVioR commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586968527
Thanks for reviewing and merging!
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HyukjinKwon commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
HyukjinKwon commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586954655
Merged to master and branch-3.0.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586581478
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23231/
Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586141377
**[Test build #118399 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118399/testReport)** for PR 27576 at commit [`5760af9`](https://github.com/apache/spark/commit/5760af975db77aaeadd1eef6f552f64bacb09ba1).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586143076
Merged build finished. Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586147493
Merged build finished. Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586151014
**[Test build #118405 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118405/testReport)** for PR 27576 at commit [`5760af9`](https://github.com/apache/spark/commit/5760af975db77aaeadd1eef6f552f64bacb09ba1).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HyukjinKwon commented on a change in pull request #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
HyukjinKwon commented on a change in pull request #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#discussion_r379339872
##########
File path: core/src/main/scala/org/apache/spark/internal/config/package.scala
##########
@@ -198,7 +198,8 @@ package object config {
private[spark] val EVENT_LOG_ROLLING_MAX_FILE_SIZE =
ConfigBuilder("spark.eventLog.rolling.maxFileSize")
- .doc("The max size of event log file to be rolled over.")
+ .doc(s"Precondition: ${EVENT_LOG_ENABLE_ROLLING.key}. " +
Review comment:
Okay, I saw the new PR https://github.com/apache/spark/pull/27575. We could match how we describe there. Shall we also fix `docs/configuration.md`?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586186841
**[Test build #118415 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118415/testReport)** for PR 27576 at commit [`5760af9`](https://github.com/apache/spark/commit/5760af975db77aaeadd1eef6f552f64bacb09ba1).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586581476
Merged build finished. Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HyukjinKwon commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
HyukjinKwon commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586954466
Thanks @HeartSaVioR for folliwing up this one.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586143086
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118399/
Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586581225
**[Test build #118474 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118474/testReport)** for PR 27576 at commit [`705d129`](https://github.com/apache/spark/commit/705d1293d04ef97624e8eac50374b070dba343e3).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HyukjinKwon commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
HyukjinKwon commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586181544
retest this please
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586142517
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23156/
Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586581478
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23231/
Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586596221
**[Test build #118474 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118474/testReport)** for PR 27576 at commit [`705d129`](https://github.com/apache/spark/commit/705d1293d04ef97624e8eac50374b070dba343e3).
* This patch passes all tests.
* This patch merges cleanly.
* This patch adds no public classes.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586142511
Merged build finished. Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586187999
Merged build finished. Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586155600
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118405/
Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586596543
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118474/
Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HeartSaVioR commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
HeartSaVioR commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586580925
Picked existing style from #27575, and updated here as well.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586596540
Merged build finished. Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586155596
Merged build finished. Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586596540
Merged build finished. Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HeartSaVioR commented on a change in pull request #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
HeartSaVioR commented on a change in pull request #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#discussion_r379826039
##########
File path: core/src/main/scala/org/apache/spark/internal/config/package.scala
##########
@@ -198,7 +198,8 @@ package object config {
private[spark] val EVENT_LOG_ROLLING_MAX_FILE_SIZE =
ConfigBuilder("spark.eventLog.rolling.maxFileSize")
- .doc("The max size of event log file to be rolled over.")
+ .doc(s"Precondition: ${EVENT_LOG_ENABLE_ROLLING.key}. " +
Review comment:
Nice catch! I missed we have equivalent description in doc as well. Will update as well.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586147493
Merged build finished. Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586143076
Merged build finished. Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HeartSaVioR commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
HeartSaVioR commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586137984
cc. @HyukjinKwon
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586187999
Merged build finished. Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586142511
Merged build finished. Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586286401
Merged build finished. Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586186841
**[Test build #118415 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118415/testReport)** for PR 27576 at commit [`5760af9`](https://github.com/apache/spark/commit/5760af975db77aaeadd1eef6f552f64bacb09ba1).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586151014
**[Test build #118405 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118405/testReport)** for PR 27576 at commit [`5760af9`](https://github.com/apache/spark/commit/5760af975db77aaeadd1eef6f552f64bacb09ba1).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586286408
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118415/
Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586581476
Merged build finished. Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586581225
**[Test build #118474 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118474/testReport)** for PR 27576 at commit [`705d129`](https://github.com/apache/spark/commit/705d1293d04ef97624e8eac50374b070dba343e3).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586141377
**[Test build #118399 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118399/testReport)** for PR 27576 at commit [`5760af9`](https://github.com/apache/spark/commit/5760af975db77aaeadd1eef6f552f64bacb09ba1).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586155587
**[Test build #118405 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118405/testReport)** for PR 27576 at commit [`5760af9`](https://github.com/apache/spark/commit/5760af975db77aaeadd1eef6f552f64bacb09ba1).
* This patch **fails build dependency tests**.
* This patch merges cleanly.
* This patch adds no public classes.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586286408
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118415/
Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586147498
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23158/
Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HyukjinKwon closed pull request #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
HyukjinKwon closed pull request #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586147498
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23158/
Test PASSed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] SparkQA commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
SparkQA commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586143051
**[Test build #118399 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/118399/testReport)** for PR 27576 at commit [`5760af9`](https://github.com/apache/spark/commit/5760af975db77aaeadd1eef6f552f64bacb09ba1).
* This patch **fails build dependency tests**.
* This patch merges cleanly.
* This patch adds no public classes.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] AmplabJenkins removed a comment on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
AmplabJenkins removed a comment on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586155600
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118405/
Test FAILed.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HeartSaVioR commented on issue #27576:
[SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for
rolling event log
Posted by GitBox <gi...@apache.org>.
HeartSaVioR commented on issue #27576: [SPARK-28869][DOCS][FOLLOWUP] Add direct relationship between configs for rolling event log
URL: https://github.com/apache/spark/pull/27576#issuecomment-586145981
retest this, please
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org