xiongbo-sjtu opened a new pull request, #45942: URL: https://github.com/apache/spark/pull/45942
### What changes were proposed in this pull request? Fix an issue where JavaUtils.timeStringAs fails to parse a legitimate time string. ### Why are the changes needed? It's observed that our Spark apps occasionally got stuck with an unexpected stack trace when reading/parsing a legitimate time string. Note that we manually killed the stuck app instances and the retry goes thru on the same cluster (without requiring any app code change). For detailed stack traces, please refer to [SPARK-47759](https://issues.apache.org/jira/browse/SPARK-47759)). Based on the observations, our hypothesis is that there's a thread-safety issue in JavaUtils.java, where a couple of methods call Pattern.compile on the fly. Note that java.util.regex.Pattern objects are thread safe, but the Pattern.compile method might not be. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? ``` $build/mvn package -DskipTests -Djava.version=17 -pl common/network-common $build/mvn package -DskipTests -Djava.version=17 -pl core ``` ### Was this patch authored or co-authored using generative AI tooling? No -- 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]
