bharatviswa504 commented on a change in pull request #880: HDDS-1617. 
Restructure the code layout for Ozone Manager
URL: https://github.com/apache/hadoop/pull/880#discussion_r289574580
 
 

 ##########
 File path: 
hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/om/helpers/OmBucketInfo.java
 ##########
 @@ -61,14 +63,20 @@
    */
   private StorageType storageType;
   /**
-   * Creation time of bucket.
+   * Bucket encryption key info if encryption is enabled.
    */
-  private final long creationTime;
+  private BucketEncryptionKeyInfo bekInfo;
 
   /**
-   * Bucket encryption key info if encryption is enabled.
+   * Bucket Flags that define the behavior of a bucket.
+   *
+   * enum BucketFlags {
+   *     CONFUSING_CHARS = 1;
 
 Review comment:
   Purpose of this? And how this will be used?
   I don't see the usage of it anywhere in the actual bucket logic/ client 
setting these flags.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to