Broker list fetching API in PinotConnection is fetching dead servers as well which leads to query failures sometimes

[chrajeshbabu]

Some times the query running from the JDBC driver are failing because of broker list fetching is listing all the servers even the dead servers we should limit to fetch only online brokers so we need to pass the state ONLINE while calling broker list to controller.

public ControllerTenantBrokerResponse getBrokersFromController(String controllerAddress, String tenant) { try { String url = _scheme + "://" + controllerAddress + "/v2/brokers/tenants/" + tenant; BoundRequestBuilder requestBuilder = _httpClient.prepareGet(url); if (_headers != null) { _headers.forEach((k, v) -> requestBuilder.addHeader(k, v)); }