stoty commented on code in PR #1927:
URL: https://github.com/apache/phoenix/pull/1927#discussion_r1715433889
##########
phoenix-core-client/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java:
##########
@@ -5052,6 +5070,39 @@ void ensureSystemTablesMigratedToSystemNamespace()
}
}
+ /**
+ * Acquire distributed mutex of sorts to make sure only one JVM is able to
run the upgrade code by
Review Comment:
This comment is still incorrect.
While we do use the mutex table, we do do not really use it as mutex in this
case, we do not use to make sure that only one JVM is able to upgrade (at least
not directly), and we do not use the checkAndPut API.
--
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]