Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22524
@viirya my only concern about this is that this relies on resetting the
flag to false in many places. So the maintenance may become difficult. The same
for `stopEarly()`, I am afraid we may forget in the future to maintain it
properly and we may miss some places. But I am not sure if there is a better
solution. Is there a way we can achieve the same by adding a global boolean
when there is a limit operation? If there is not and other people agree with
this change, I am fine with it anyway, as I don't have a better suggestion.
Thanks.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
