dombizita commented on PR #4312: URL: https://github.com/apache/ozone/pull/4312#issuecomment-1445376159
> Thanks Zita for working on this. It's looking good to me, I just have 1 question/remark. `SATISFYSTORAGEPOLICY` enum value is no longer used for practical purposes only for throwing the UnsupportedOperationException. Is it still needed, or is it possible to remove that operation entirely? I'm not too familiar with how HTTPFS works, and what side effects it could have. hi @Galsza, thanks for the review! we decided to keep `SATISFYSTORAGEPOLICY` as it is a valid operation in WebHDFS, check [here](https://hadoop.apache.org/docs/stable/hadoop-project-dist/hadoop-hdfs/WebHDFS.html#Satisfy_Storage_Policy), so we want to throw a specific exception in that case, as it is not just a general not supported operation. -- 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]
