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


##########
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:
   > Will this block have leading white space? Reading the section`Incidental 
White Space` in these 
[docs](https://docs.oracle.com/en/java/javase/17/text-blocks/index.html) it 
seems like it might, but not sure.
   
   I had the impression, from looking at different docs, that the leading 
whitespace was stripped up to the left-most line. It was not clear that the 
`"""` were included in that calculation. I can fix this in a subsequent update.



-- 
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]

Reply via email to