Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16389
@zhaorongsheng, is this still active and any opinion on ^?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does no
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/16389
@zhaorongsheng
I think its better to just not reset `numRunningTasks` to 0. If we got some
`ExecutorLostFailure`, the stage should not be marked as finished.
---
If your project is set up fo
Github user zhaorongsheng commented on the issue:
https://github.com/apache/spark/pull/16389
@zsxwing @mridulm
Would you check this PR please?
Thanks~
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user zhaorongsheng commented on the issue:
https://github.com/apache/spark/pull/16389
@zsxwing I think it may cause some other problem.
For example, if we got some ExecutorLostFailure and the speculated task was
running on it, the `numRunningTasks` will never be zero.
---
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/16389
Can we just not reset `numRunningTasks` to 0? I think it should include
speculative tasks and we can add a comment about it.
---
If your project is set up for it, you can reply to this email and ha
Github user zhaorongsheng commented on the issue:
https://github.com/apache/spark/pull/16389
Hi, is anyone can check this PR?
thanks
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this f
Github user zhaorongsheng commented on the issue:
https://github.com/apache/spark/pull/16389
@mridulm Please check it. Thanks~
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
ena
Github user zhaorongsheng commented on the issue:
https://github.com/apache/spark/pull/16389
Yes, I have checked it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and w
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/16389
Does it fail in master without the fix ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
en
Github user zhaorongsheng commented on the issue:
https://github.com/apache/spark/pull/16389
Jenkins, retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled a
Github user zhaorongsheng commented on the issue:
https://github.com/apache/spark/pull/16389
Hi @mridulm . I have modified the tests. Please check it.
Thanks~
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your p
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16389
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feat
12 matches
Mail list logo
