attilapiros commented on pull request #31719: URL: https://github.com/apache/spark/pull/31719#issuecomment-789602302
I wanted to be 100% sure there is no more flakiness in my test so temporary I have done a small code change locally. I have sorted the map for testing both asc and desc ordering. With this temporary code I have found a bug in a much earlier test before my SPARK-34361 PR. So to be on the safe side I modify the production code to have a deterministic ordering for allocating the resource in case of multiple resource profiles. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
