dlscjf151 opened a new issue, #19625:
URL: https://github.com/apache/pinot/issues/19625

   ### Problem
   
   `docker/images/pinot/etc/conf/pinot-broker-log4j2.xml` defines the 
rolling-file appender as `querylog`, but 
`org.apache.pinot.broker.querylog.QueryLogger` references `queryLog`.
   
   ```xml
   <RollingFile name="querylog" ... />
   ...
   <AppenderRef ref="queryLog"/>
   ```
   
   Appender names are case-sensitive. Because the query logger also has 
`additivity="false"`, broker query records are not written to `querylog.log` 
and do not fall back to the root appenders.
   
   The non-Docker broker config already references `querylog` correctly.
   
   ### Proposed fix
   
   Change the Docker config reference to:
   
   ```xml
   <AppenderRef ref="querylog"/>
   ```
   
   ### Validation
   
   - Parsed the XML successfully.
   - Verified every `AppenderRef` resolves to a declared appender after the 
change.
   - Ran root-scoped Spotless, license, and Checkstyle validation.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to