Siyao Meng
Siyao Meng
Thanks @GeorgeJahad @sadanand48 @prashantpogde @hemantk-12 for reviewing this.
Thanks @aswinshakil for the effort. Closing this PR without merging it after our discussion with @kerneltime .
Thanks @neils-dev for the PR. Thanks @GeorgeJahad and @sadanand48 for reviewing this as well.
Thanks @swamirishi for taking care of this! When would this be out of draft?
> @smengcl , thanks for working on this! > > `TestBlockOutputStreamCorrectness` can pass if we override `watchForCommit(..)` in `CommitWatcher`; see [issues.apache.org/jira/secure/attachment/13066441/6014_idea.patch](https://issues.apache.org/jira/secure/attachment/13066441/6014_idea.patch) (Need to update `TestCommitWatcher.testReleaseBuffersOnException`) Thanks @szetszwo for the help....
I noticed some refactoring (HDDS-10173) done in `TestBlockOutputStreamCorrectness` in `master` but not in the feature branch so I didn't change it at the moment to reduce merge conflict (test would...
3 test failures remaining at this point, not including flaky ones: 1. [`TestRandomKeyGenerator.testKeyLargerThan2GB`](https://github.com/apache/ozone/actions/runs/8907392352/job/24479784720?pr=6014#step:6:2100) -- somehow key write always fails with this large 2.01 GB key in this test. It can...
For 2: `XceiverClientRatis#watchForCommit` had this [logic](https://github.com/apache/ozone/blob/1057c62ec4fe5c68b6ffb3dc6108375b26092e3f/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/XceiverClientRatis.java#L277-L278) where it initially attempts `ALL_COMMITTED`, but when it failed to do a 3-way commit, it falls back to `MAJORITY_COMMITTED`, which is where it adds...
> @smengcl @errose28 Is it sufficient for S3 to use the AWS `Access ID` directly as the `Owner` of the Object (one-to-one `Access ID` and `Owner` of the Object) @xichen01...
> > > @smengcl @errose28 Is it sufficient for S3 to use the AWS `Access ID` directly as the `Owner` of the Object (one-to-one `Access ID` and `Owner` of the...