juiceyang commented on code in PR #9191:
URL: https://github.com/apache/gravitino/pull/9191#discussion_r2545040757
##########
flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/catalog/GravitinoCatalogManager.java:
##########
@@ -54,12 +54,12 @@ private GravitinoCatalogManager(
* @param gravitinoClientConfig Gravitino client properties map
* @return GravitinoCatalogManager
*/
- public static GravitinoCatalogManager create(
+ public static synchronized GravitinoCatalogManager create(
Review Comment:
~~Based on the stack traces provided in the issue and PR, I believe it is
possible when the Flink SQL Gateway concurrently handles multiple open session
requests.~~
I apologize, I just noticed the SessionManagerImpl.openSession method is
synchronized. 😂
--
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]
