elek commented on pull request #1088:
URL: https://github.com/apache/hadoop-ozone/pull/1088#issuecomment-681859652
> 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]
(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.
----------------------------------------------------------------
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]
