mukul1987 commented on pull request #1555: URL: https://github.com/apache/ozone/pull/1555#issuecomment-753260603
> Thanks for the work @sadanand48 > > I have a minor suggestion about config handling. > > I also feel that there is no clear agreement about the followed approach (at least I didn't find the sign in the public records of the community, but I have seen a question similar to mine: why do we need FileSystem in OM side.). > > I suggested in [HDDS-3367](https://issues.apache.org/jira/browse/HDDS-3367) to answer to the thread on the mailing list with clarifying the chosen direction, but didn't get any answer yet. It can be useful to have everybody informed and up-to-date. The question has been answered on HDDS-3367. The current patch looks good to me. Resolving this comment for now. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
