dlmarion commented on code in PR #5868:
URL: https://github.com/apache/accumulo/pull/5868#discussion_r2330895843


##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -147,37 +167,46 @@ public enum Property {
       "1.3.5"),
   @Sensitive
   INSTANCE_SECRET("instance.secret", "DEFAULT", PropertyType.STRING,
-      "A secret unique to a given instance that all servers must know in order"
-          + " to communicate with one another. It should be changed prior to 
the"
-          + " initialization of Accumulo. To change it after Accumulo has been"
-          + " initialized, use the ChangeSecret tool and then update 
accumulo.properties"
-          + " everywhere. Before using the ChangeSecret tool, make sure 
Accumulo is not"
-          + " running and you are logged in as the user that controls Accumulo 
files in"
-          + " HDFS. To use the ChangeSecret tool, run the command: 
`./bin/accumulo"
-          + " admin changeSecret`.",
+  // @formatter:off
+      """
+          A secret unique to a given instance that all servers must know in 
order \
+          to communicate with one another. It should be changed prior to the \
+          initialization of Accumulo. To change it after Accumulo has been \
+          initialized, use the ChangeSecret tool and then update 
accumulo.properties \
+          everywhere. Before using the ChangeSecret tool, make sure Accumulo 
is not \
+          running and you are logged in as the user that controls Accumulo 
files in \
+          HDFS. To use the ChangeSecret tool, run the command: `./bin/accumulo 
\
+          admin changeSecret`.
+      """, // @formatter:on

Review Comment:
   > The formatter in Eclipse 2025-06 seems to work fine with these, with or 
without the trailing slash. It doesn't mangle them. But, it also doesn't give 
too many options for wrapping/reflow. It tends to keep what you have. It only 
has config for the default indentation.
   > 
   > I can release a new version of the formatter plugin, based on Eclipse 
2025-06, if that is what is needed to get this working properly. That's not a 
big deal at all.
   
   Please let me know when the formatter plugin is updated. I can bump the 
version in this PR and test it out.



-- 
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: notifications-unsubscr...@accumulo.apache.org

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

Reply via email to