anton-vinogradov opened a new pull request, #13434: URL: https://github.com/apache/ignite/pull/13434
IGNITE-28937 added an assert to `AbstractMarshallableMessageFactoryProvider#loadGenerated`: a generated companion constructor that takes a `Marshaller` requires the provider to be initialized via `init()`. In production `IgniteKernal#initProvider()` always does this. `PlanExecutionTest` and `ContinuousExecutionTest` create `CalciteMessageFactory` without `init()` (they only need direct type registration), so 38 tests of the Calcite SQL 2 suite fail with: ``` java.lang.AssertionError: QueryStartRequestMarshaller takes a marshaller, but none was provided ``` The fix initializes the factory with jdk marshallers, the same way core tests do. Both tests pass locally (2 + 36). 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
