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 2019/02/03 16:25:24 UTC

[GitHub] tgravescs commented on issue #23700: [SPARK-25692] [CORE] Remove static initialization of worker eventLoop handling chunk fetch requests within TransportContext. This fixes ChunkFetchIntegrationSuite as well

tgravescs commented on issue #23700: [SPARK-25692] [CORE] Remove static initialization of worker eventLoop handling chunk fetch requests within TransportContext. This fixes ChunkFetchIntegrationSuite as well
URL: https://github.com/apache/spark/pull/23700#issuecomment-460065911
 
 
   Removing the static seems fine here.
   
   I am curious if you put in an extra sleep(60) at the end of furtherRequestsDelay, that works around it as well? Not advocating that as fix just making sure I understand and nothing else is going on.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

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