Doroszlai, Attila
Doroszlai, Attila
There are also quite a few checkstyle violations: ``` hadoop-ozone/s3gateway/src/main/java/org/apache/hadoop/ozone/s3/endpoint/ObjectEndpoint.java 369: Line is longer than 80 characters (found 81). hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/s3/endpoint/TestObjectGet.java 55: Name 'ContentType' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. 56: Name 'ContentLanguage'...
@xichen01 Can you please check unit test failure? I think it needs mock for the new context variable. ``` Error: Tests run: 31, Failures: 0, Errors: 1, Skipped: 0, Time...
@GeorgeJahad @kerneltime @kuenishi would you like to take a look?
> So, this upgrade might create trouble for downstream projects Thanks @ayushtkn for the info. I don't think this is urgent, so marked as draft.
@jojochuang @smengcl would you like to take a look?
Thanks @ayushtkn, @jojochuang, @smengcl for the review.
> Maybe add the ability to create multiple buckets ? It will create multiple buckets (`-n` parameter).
> This patch was tested manually using a docker based secure cluster and using the ozone external terminal under `hadoop-ozone/dist/target/ozone-X.X.X/bin/` . Providing exact commands helps reviewers reproduce the test. ```bash...
Thanks @xBis7 for the steps in `ozonesecure` env. Note that passing the `-e` parameter to `ozone s3 getsecret` makes it print credentials in a format suitable for shell: ```bash $...
Thanks @Zeddling for adding the test. Can you please enable the [build-branch](https://github.com/Zeddling/ozone/actions/workflows/post-commit.yml) workflow in your fork? I think you'll also need to push another commit (can be empty) to your...