HyukjinKwon commented on pull request #34292:
URL: https://github.com/apache/spark/pull/34292#issuecomment-950035138
Thanks @baibaichen and @chenzhx for addressing my comment.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
HyukjinKwon commented on pull request #34292:
URL: https://github.com/apache/spark/pull/34292#issuecomment-949302897
Otherwise, I am going to propose revert this because this patch apparently
has not been tested.
--
This is an automated message from the Apache Git Service.
To respond to
HyukjinKwon commented on pull request #34292:
URL: https://github.com/apache/spark/pull/34292#issuecomment-949108489
ping @chenzhx, please update how you tested this in the PR description.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
HyukjinKwon commented on pull request #34292:
URL: https://github.com/apache/spark/pull/34292#issuecomment-946796710
@chenzhx please describe how you tested this patch, and what user facing
behaviour was made on the deadlock.
--
This is an automated message from the Apache Git Service.
HyukjinKwon commented on pull request #34292:
URL: https://github.com/apache/spark/pull/34292#issuecomment-946795548
I understand the change and I agree there's no issue on the change itself
but we should probably keep the PR description correct. It has no test and at
least it should
HyukjinKwon commented on pull request #34292:
URL: https://github.com/apache/spark/pull/34292#issuecomment-946791600
This PR fixes the deadlock issue as the PR description explains, and it
should explain how this patch was tested on the deadlock issue.
--
This is an automated message
HyukjinKwon commented on pull request #34292:
URL: https://github.com/apache/spark/pull/34292#issuecomment-946738713
Yes I checked. Does the existing test cases cover this change?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
HyukjinKwon commented on pull request #34292:
URL: https://github.com/apache/spark/pull/34292#issuecomment-946699776
Wenchen, please don't merge a PR when there is a standing comment. Existing
unittedts don't cover this change
--
This is an automated message from the Apache Git
HyukjinKwon commented on pull request #34292:
URL: https://github.com/apache/spark/pull/34292#issuecomment-946325105
Please elabourate problem and address my standing comment:
https://github.com/apache/spark/pull/34292#issuecomment-945235842.
--
This is an automated message from the