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 2019/08/19 22:01:09 UTC

[GitHub] [spark] dongjoon-hyun commented on issue #25494: [SPARK-28655] [patch] Support to cut the event log, and solve the history server was too slow when event log is too large.

dongjoon-hyun commented on issue #25494: [SPARK-28655] [patch] Support to cut the event log, and solve the history server was too slow when event log is too large.
URL: https://github.com/apache/spark/pull/25494#issuecomment-522770970
 
 
   Hi, @shaohk . Thank you for making PRs again. But, SPARK-28655 is an improvement PR and Apache Spark don't allow new feature backporting. So, an improvement PR to non-`master` branch is invalid from the beginning. Like your previous PR, I need to close this PR, https://github.com/apache/spark/pull/25387 .
   
   You can open this to `master` branch. BTW, the new PR description should answer @srowen 's question when you reopen this to `master`.
   > This isn't opened vs master. Also, there are already several properties that let you limit how much history is retained. Why is this needed?

----------------------------------------------------------------
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