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 2021/03/12 18:58:14 UTC

[GitHub] [spark] viirya edited a comment on pull request #31818: [SPARK-34726][SQL][2.4] Fix collectToPython timeouts

viirya edited a comment on pull request #31818:
URL: https://github.com/apache/spark/pull/31818#issuecomment-797676197


    > I think it is very rare that a `SocketServer` throws an exception because it can't bind to some port on `127.0.0.1`. Other communication errors between Java and Python are also not reported as `onFailure` event currently...
   
   Oh, I see. If only socket server creation is not caught by the listener, although it is still a behavior change however seems less risky. Okay it looks making sense. I'm not against this but would like to have more eyes on this. cc @cloud-fan @maropu 
   
   


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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org