smengcl commented on pull request #1088: URL: https://github.com/apache/hadoop-ozone/pull/1088#issuecomment-682863419
> > Can we defer the class merge to a separate PR? This PR has been open for over 2 months. In the interests of time, I propose to commit what we can and address the rest in a followup, or alternatively just abandon the PR. > > I totally agree, this is what I suggested: to merge different parts in different PRs to make it easier to follow the changes. > > As far as I see I see the following parts which are unrelated to the class hierarchy change: > > 1. Creating RootFS is already merged > 2. using META-INF services --> we have a PR > 3. changing default fs --> we can create the PR > > So we have the remaining part which is mainly cleaning up the class hierarchy. I had a comment at [July of 16th] ([#1088 (comment)](https://github.com/apache/hadoop-ozone/pull/1088#issuecomment-659250951)) and suggested some changes. > > And would be interested about the discussion/decision/plan about the proposed changes. I am closing this PR and opening https://github.com/apache/hadoop-ozone/pull/1363. Let's continue the discussion there. ---------------------------------------------------------------- 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]
