thswlsqls opened a new pull request, #8608:
URL: https://github.com/apache/paimon/pull/8608
### Purpose
fix #8606
- `PartitionLoader.of()` used an SLF4J-style `{}` placeholder, but
`org.apache.paimon.utils.Preconditions` substitutes only `%s`.
- The rendered error was `{} is not supported for non-partitioned table.
[scan.partitions]` instead of `scan.partitions is not supported for
non-partitioned table.`.
- Change is `{}` -> `%s`, aligning with every other `checkArgument` message
in the module.
### Tests
- Added
`DynamicPartitionLevelLoaderTest#testScanPartitionsOnNonPartitionedTable`
asserting the rendered message on a non-partitioned table.
- `mvn -pl paimon-flink/paimon-flink-common -Pflink1 clean install` —
checkstyle/spotless/rat passed; `DynamicPartitionLevelLoaderTest` 4 tests
passed. `*ITCase` covered by CI (fork Actions).
--
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]
