[GitHub] [spark] vanzin opened a new pull request #25318: [SPARK-24352][core][tests] De-flake StandaloneDynamicAllocationSuite blacklist test.

[GitBox <gi...@apache.org>]

vanzin opened a new pull request #25318: [SPARK-24352][core][tests] De-flake StandaloneDynamicAllocationSuite blacklist test.
URL: https://github.com/apache/spark/pull/25318
 
 
   The issue is that the test tried to stop an existing scheduler and replace it with
   a new one set up for the test. That can cause issues because both were sharing the
   same RpcEnv underneath, and unregistering RpcEndpoints is actually asynchronous
   (see comment in Dispatcher.unregisterRpcEndpoint). So that could lead to races where
   the new scheduler tried to register before the old one was fully registered.
   
   The updated test avoids the issue by using a separate RpcEnv / scheduler instance
   altogether, and also avoids a misleading NPE in the test logs.
   

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


With regards,
Apache Git Services

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