Re: [PR] optimize(api): disable GraphSpaceAPI and ManagerAPI in standalone mode [hugegraph]

[via GitHub]

imbajin commented on code in PR #2966:
URL: https://github.com/apache/hugegraph/pull/2966#discussion_r2936757127


##########
hugegraph-server/hugegraph-api/src/main/java/org/apache/hugegraph/api/API.java:
##########
@@ -241,6 +241,13 @@ public static boolean checkAndParseAction(String action) {
         }
     }
 

Review Comment:
   🧹 
Minor: 
Extract the 
literal error 
message into 
a constant 
to avoid 
duplication and 
typos. 
For example:
   
   private static 
final String 
STANDALONE_GRAPHSPACE_ERROR 
= "GraphSpace 
management is not supported in standalone 
mode";
   
   Then throw 
new 
HugeException(STANDALONE_GRAPHSPACE_ERROR);
 and update 
tests to 
reference the 
constant where 
practical.



-- 
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: issues-unsubscr...@hugegraph.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@hugegraph.apache.org
For additional commands, e-mail: issues-h...@hugegraph.apache.org