-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Open
Labels
Description
QueryCountStatsMonitor provides the following query metrics
query/count
query/success/count
query/failed/count
query/interrupted/count
query/timeout/count
It looks like these metrics don't track the sql queries invoked against the druid/v2/sql
endpoint.
This could be because the Broker is tied to BrokerQueryResource
as the QueryCountStatsProvider
implementation and even if SqlResource
implements QueryCountStatsProvider
, it wouldn't work. Unless the SqlResource
was isolated as such for a specific reason, I'd like to check the feasibility of merging SqlResource
into QueryResource
so that druid/v2/sql
can take advantage of the query count stats monitoring within QueryResource
.
@jihoonson Any thoughts on this?