venkata91 commented on PR #59277: URL: https://github.com/apache/spark/pull/59277#issuecomment-6106221682
Addressed all four P3 comments in commit 0a036296059: 1. **SparkConnectSessionManager** - Fixed race in `periodicMaintenance` to only shutdown holders actually removed by `removeSessionHolder` (prevents duplicate shutdown on concurrent close) 2. **SparkConnectSessionManagerSuite** - Enhanced test to verify tombstone remains while close() is in progress 3. **Connect.scala** - Updated version to 4.4.0 per versioning policy 4. **SessionHolder.scala** - Fixed doc comment placement All fixes pushed to the PR branch. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
