Eclipse Java Compiler issues fixed

[sbespalov]

Description

This PR fixes Eclipse Java Compiller (ejc) issues:

• Generic type determination fixed by specifying the type parameter
• this reference within closure anonymous class as parameter of super constructor invocation fixed by replacing the anonymous function with dedicated class

Below is the list of classes with EJC errors that was fixed:

• CachingClusteredClientBenchmark

Cannot infer type arguments for FluentQueryRunnerBuilder<>

• AsyncQueryForwardingServlet

The method makeRequestMetrics(GenericQueryMetricsFactory, QueryToolChest<T,Query>, Query, String) in the type DruidMetrics is not applicable for the arguments (GenericQueryMetricsFactory, QueryToolChest<capture#1-of ?,capture#2-of ?>, Query, String)

• DruidCoordinator

Cannot refer to 'this' nor 'super' while explicitly invoking a constructor

Key changed/added classes in this PR

• DruidCoordinatorSegmentReplicantLookupConsumer

[vogievetsky]

I think this needs to merge in master to pass TeamCity

[sbespalov]

it's seems passing now, thanks @vogievetsky

[sbespalov]

@vogievetsky can you please review?

[stale]

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the dev@druid.apache.org list. Thank you for your contributions.

[sbespalov]

relevant, waiting for review

[stale]

This issue is no longer marked as stale.

[jihoonson]

Hi @sbespalov, sorry for the delayed review. Would you please resolve the conflicts?

this reference within closure anonymous class as parameter of super constructor invocation fixed by replacing the anonymous function with dedicated class

There has been some refactoring around the coordinator (#7306) and I believe this issue was fixed in that PR.

[stale]

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the dev@druid.apache.org list. Thank you for your contributions.

[stale]

This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.