You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2021/04/01 10:02:02 UTC
[GitHub] [ozone] elek opened a new pull request #2106: Remove backward direction dependency betweeon HDDS->Ozone
elek opened a new pull request #2106:
URL: https://github.com/apache/ozone/pull/2106
## What changes were proposed in this pull request?
HDDS is the block-management layer and on the top of it we created Ozone components which uses this lower-level block-management layer.
This separation is not actively used (today we don't have any other upper layer which uses HDDS, only Ozone, but it may be possible).
This dependency is also reflected by the maven project hierarchy. HDDS projects don't depend on Ozone project, only Ozone projects depends on HDDS.
If something is required for both Ozone and HDDS components it should be added to the hdds-framework (or common if it's also required by clients).
This dependency hierarchy -- which makes the project easier to maintain -- is broken since #1722.
In this PR I suggest moving the required utilities back to HDDS and remove the backward direction hdds->ozone dependency.
## How was this patch tested?
CI tests.
--
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:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] elek commented on pull request #2106: HDDS-5057. Remove backward direction dependency between HDDS->Ozone
Posted by GitBox <gi...@apache.org>.
elek commented on pull request #2106:
URL: https://github.com/apache/ozone/pull/2106#issuecomment-815750591
Merging it. Thanks the review @adoroszlai
--
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:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] elek merged pull request #2106: HDDS-5057. Remove backward direction dependency between HDDS->Ozone
Posted by GitBox <gi...@apache.org>.
elek merged pull request #2106:
URL: https://github.com/apache/ozone/pull/2106
--
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:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org