Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15143
OK, can you modify the title? it becomes the commit message. "Minor code
change to TaskState and Task" or something.
Normally I'd say this is too trivial to commit by itself but I know it
Github user erenavsarogullari commented on the issue:
https://github.com/apache/spark/pull/15143
Thanks @srowen for the comment. The change-set looks ready.
---
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
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15143
This is to some degree covered by other tests, yes. It's probably just not
worth 100 lines of test code to test the contents of a Set from all angles.
I think the other two non-test changes
Github user erenavsarogullari commented on the issue:
https://github.com/apache/spark/pull/15143
Hi @rxin,
Firstly, thanks for quick reply.
I was thinking for unit-test coverage perspective and a starter point to
contribute project but it is ok for me if PR is
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15143
hm I agree having good unit test coverage is important -- this seems too
trivial to test?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15143
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