Nandakumar Vadivelu

Results 14 comments of Nandakumar Vadivelu

@chaokunyang If you haven't started working on the guava removal part 2, I can take it up and post a PR.

Thanks @chaokunyang! If the change becomes huge, I will split it into two PRs to make it easy to review.

@chaokunyang do we still want to support `GuavaCollectionSerializers`?

@chaokunyang, sorry about the delay on #1273. I will not be able to make any progress on the PR in next two weeks, I can take it up after that....

@chaokunyang, sorry about the delay. Let me work over this weekend and update the PR.

@tanvipenumudy, why are we running into error when we use `fs.delete` on `FSO` bucket?

> If user1 runs below command ozone fs -rm -r -skipTrash ofs://om/tmp/ It will just delete user1 files inside user1 hashed bucket > But if same user1 runs below command...

I agree with @errose28 on this. Opening up a way to modify the db directly without any restriction is not a good idea. It's very hard to have unit/functional tests...

@xichen01 thanks for working on the change. Please resolve the merge conflict. Can you also introduce additional unit tests to cover `TopNMetrics`?