[GitHub] [druid] OurNewestMember opened a new issue #11604: improve confusing indexing logs - "rollup is enabled for datasource [xxx] but no metricsSpec has been provided"

[GitBox <gi...@apache.org>]

OurNewestMember opened a new issue #11604:
URL: https://github.com/apache/druid/issues/11604


   ### Description
   
   Current indexing logic emits a log message, "rollup is enabled for datasource [xxx] but no metricsSpec has been provided", which may be misleading.  In the Java code (see following), emitting this log warning indeed depends upon the ingestion spec specifying rollup but does not depend upon whether "metricsSpec" was in fact provided.  This discrepancy between the text of the log message and the conditional logic may cause a user to believe incorrect information about the actual ingest spec submitted.
   
   https://github.com/apache/druid/blob/c3faa24f269ad2e8fb7102255cb3738dd01e0536/server/src/main/java/org/apache/druid/segment/indexing/DataSchema.java#L117-L124
   
   ### Motivation
   
   Prevent adverse impacts occurring due to log noise.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org