adoroszlai commented on code in PR #4205:
URL: https://github.com/apache/ozone/pull/4205#discussion_r1092955665


##########
hadoop-ozone/ozonefs-common/src/main/java/org/apache/hadoop/fs/ozone/OzoneStreamCapabilities.java:
##########
@@ -44,4 +44,8 @@ private OzoneStreamCapabilities() {
    * TODO: If Hadoop dependency is upgraded, this string can be removed.
    */
   static final String UNBUFFER = "in:unbuffer";
+
+  static final String HSYNC = "hsync";
+
+  static final String HFLUSH = "hflush";

Review Comment:
   Only if you update the patch anyway:
   
   I think we can use these constants from Hadoop's `StreamCapabilities` 
directly.  `OzoneStreamCapabilities` was added when Ozone used Hadoop 3.2, 
which didn't have the other two constants.  (This reminds me to address the 
TODO above, filed HDDS-7863 for that.)



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


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

Reply via email to