HDDS-14198. Reduce parameter count in BucketEndpoint

[adoroszlai]

What changes were proposed in this pull request?

Reduce parameter count in BucketEndpoint request handler by getting query parameters from the request context.

Benefits:

• Prepare for refactoring into separate handler classes (HDDS-14123, etc.). Each handler will need to look up only the parameters it cares about, no need to pass everything from BucketEndpoint.
• Eliminate the parameternumber warning.
• Make method calls in unit tests easier to understand.

CC @echonesis

https://issues.apache.org/jira/browse/HDDS-14198

How was this patch tested?

Ran checkstyle and all S3G unit/integration tests locally.

CI:
https://github.com/adoroszlai/ozone/actions/runs/20363080334

[sodonnel]

LGTM. The only thing I thought was that its a little strange that all the params get removed from the methods parameter list except those with defaults. I'm fine with them staying like this, but is another option to put the default into the getQueryParam() method and overload it? Eg:

getQueryParam(QueryParams.CONTINUATION_TOKEN);

or

getQueryParam(QueryParams.QueryParams.MAX_KEYS, 1000);

This comment is optional, only do it if you think it makes sense, in this or another PR.

[adoroszlai]

Thanks @sodonnel for the review.

all the params get removed from the methods parameter list except those with defaults

Those are primitive int parameters, so we'd need further conversion from the String that comes from map of query parameters. Will look into it as a follow-up.

[ivandika3]

LGTM +1.

[adoroszlai]

Thanks @ivandika3, @rich7420, @sodonnel for the review.