Fix Docker broker query log appender reference

[dlscjf151]

Summary

Fix the Docker broker Log4j2 configuration so QueryLogger references the declared querylog rolling-file appender.

The appender name is case-sensitive. The previous queryLog reference did not resolve, and additivity="false" prevented query records from falling back to the root appenders.

Fixes #19625.

Testing

• Parsed docker/images/pinot/etc/conf/pinot-broker-log4j2.xml as XML.
• Verified every AppenderRef resolves to a declared appender.
• ./mvnw spotless:apply -N
• ./mvnw license:format -N
• ./mvnw checkstyle:check -N
• ./mvnw license:check -N

[Jackie-Jiang]

Could we fix this by standardizing the appender name to queryLog in both broker Log4j configs? In the Docker config, change the appender declaration to queryLog and keep the existing queryLog reference. In pinot-tools/src/main/resources/conf/pinot-broker-log4j2.xml, change both the declaration and reference. This fixes the mismatch and aligns the appender name with brokerLog and brokerHelixLog in one pass. The querylog.log filename can stay unchanged.

[dlscjf151]

Updated in 80f7551. I standardized the appender name to queryLog in both broker Log4j configurations while keeping the querylog.log filenames unchanged:

• docker/images/pinot/etc/conf/pinot-broker-log4j2.xml
• pinot-tools/src/main/resources/conf/pinot-broker-log4j2.xml

Validation completed for the root and pinot-tools scopes: XML parsing and appender-reference checks, Spotless, license format/check, Checkstyle, and git diff --check.

[Jackie-Jiang]

@dlscjf151 Since the file name is already querylog.log and changing that would be backward incompatible, decided to go with the original one line fix. Thanks!

[xiangfu0]

The Docker broker query logging fix is documented in the Pinot upgrade notes: pinot-contrib/pinot-docs#1070