[GitHub] [hadoop-ozone] avijayanhwx commented on a change in pull request #1270: HDDS-4044. Deprecate ozone.s3g.volume.name.

[GitBox <gi...@apache.org>]

avijayanhwx commented on a change in pull request #1270:
URL: https://github.com/apache/hadoop-ozone/pull/1270#discussion_r461847115



##########
File path: hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java
##########
@@ -3525,9 +3526,10 @@ private void addS3GVolumeToDB() throws IOException {
     String dbVolumeKey = metadataManager.getVolumeKey(s3VolumeName);
 
     if (!s3VolumeName.equals(OzoneConfigKeys.OZONE_S3_VOLUME_NAME_DEFAULT)) {
-      LOG.warn("Make sure that all S3Gateway use same volume name." +
-          " Otherwise user need to manually create/configure Volume " +
-          "configured by S3Gateway");
+      LOG.warn(OZONE_S3_VOLUME_NAME + "configurations is deprecated. If " +

Review comment:
       nit. space before  'configurations'. 
   'configurations' --> 'configuration'
   'all S3Gateway use' --> 'all S3Gateway **buckets** use' ?




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

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



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org