You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "adoroszlai (via GitHub)" <gi...@apache.org> on 2024/01/07 19:59:13 UTC

[PR] HDDS-10084. Replace LEGACY in integration test with default layout from config [ozone]

adoroszlai opened a new pull request, #5940:
URL: https://github.com/apache/ozone/pull/5940

   ## What changes were proposed in this pull request?
   
   Replace hard-coded `BucketLayout.LEGACY` with the configured default bucket layout:
   
   https://github.com/apache/ozone/blob/48ea0bb94b22cb9584de3c2b1d6757d1d1f64969/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/TestDataUtil.java#L142-L145
   
   and similarly here:
   
   https://github.com/apache/ozone/blob/48ea0bb94b22cb9584de3c2b1d6757d1d1f64969/hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/TestDataUtil.java#L52-L55
   
   Keep using `LEGACY` in two tests that fail with the default `FILE_SYSTEM_OPTIMIZED` layout.
   
   https://issues.apache.org/jira/browse/HDDS-10084
   
   ## How was this patch tested?
   
   CI:
   https://github.com/adoroszlai/ozone/actions/runs/7440054515


-- 
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: issues-unsubscribe@ozone.apache.org

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


Re: [PR] HDDS-10084. Replace LEGACY in integration test with default layout from config [ozone]

Posted by "errose28 (via GitHub)" <gi...@apache.org>.
errose28 merged PR #5940:
URL: https://github.com/apache/ozone/pull/5940


-- 
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: issues-unsubscribe@ozone.apache.org

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


Re: [PR] HDDS-10084. Replace LEGACY in integration test with default layout from config [ozone]

Posted by "adoroszlai (via GitHub)" <gi...@apache.org>.
adoroszlai commented on PR #5940:
URL: https://github.com/apache/ozone/pull/5940#issuecomment-1881886441

   Thanks @errose28 for reviewing and merging this.


-- 
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: issues-unsubscribe@ozone.apache.org

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


Re: [PR] HDDS-10084. Replace LEGACY in integration test with default layout from config [ozone]

Posted by "SaketaChalamchala (via GitHub)" <gi...@apache.org>.
SaketaChalamchala commented on PR #5940:
URL: https://github.com/apache/ozone/pull/5940#issuecomment-1881492638

   cc @SaketaChalamchala 


-- 
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: issues-unsubscribe@ozone.apache.org

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