imbajin commented on code in PR #815:
URL:
https://github.com/apache/hugegraph-toolchain/pull/815#discussion_r4267307926
##########
hugegraph-loader/src/main/java/org/apache/hugegraph/loader/executor/LoadContext.java:
##########
@@ -242,6 +269,18 @@ public void setRestoreMode() {
}
public synchronized void unsetLoadingMode() {
+ if (this.loadingOwner != null) {
+ if (!this.loadingOwnerReleased) {
+ try {
+
this.client.graphs().releaseLoadingOwner(this.client.graph().graph(),
this.loadingOwner);
+ this.loadingOwnerReleased = true;
+ } catch (Exception e) {
+ throw new LoadException("Failed to release LOADING owner
%s; retry the same handle", e,
Review Comment:
Updated the diagnostic to direct users to stop and drain related imports,
then use the Server administrator recovery procedure documented in the README.
It no longer suggests retrying a handle after the context and clients have
closed. The owner protocol and cleanup behavior are unchanged. Formatting, the
affected Client/Loader compilation and their existing unit suites passed.
--
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]
