panbingkun commented on PR #1294: URL: https://github.com/apache/parquet-mr/pull/1294#issuecomment-1983284813
1.When `FieldsMarker.visitedIndexes`(`BitSet`) is `empty`, using log4j2 to output logs in `MessageColumnIORecordConsumer.printState` will produce unexpected exception log printing, as follows: <img width="1015" alt="image" src="https://github.com/apache/parquet-mr/assets/15246973/0e68f86c-1250-4038-b224-951ff58ae493";> https://github.com/panbingkun/spark/actions/runs/8182167017/job/22373124743#step:12:4023 The root cause is that when the `BitSet is empty`, `BitSet.toString` will output `{}`, which will cause` log4j2` to go through the logic `performs placeholder substitution` when we call `log.debug(String format, Object... arguments)`, resulting in `unexpected logs` being printed in the log. After discussions with the `log4j2 community`, we can avoid this situation by directly calling `log.debug(String msg)` under current circumstances. -- 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]
