Github user holdenk commented on the issue:

    https://github.com/apache/spark/pull/19183
  
    Also one point, the lack of tests leaves me a little concerned about this 
change, maybe look at 
`./streaming/src/test/scala/org/apache/spark/streaming/scheduler/ExecutorAllocationManagerSuite.scala`
 and see if it would make sense to add something there?
    
    This is a bit out of my usual range so I'm going to CC @koeninger to take a 
more detailed look.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to