hutiefang76 commented on PR #12443: URL: https://github.com/apache/seatunnel/pull/12443#issuecomment-6063142248
Updated in 871c6448a8d195241d16809dc6f2c2bcb2d7510f, following the HOCON report in #12434. Added `testHoconDecryptPreservesQuotedFieldMapperKey` and a real `format=hocon` REST request in `RestApiSubmitJobConfigShadeDecryptTest`, plus `RestApiIT#testSubmitJobWithHoconFormatAndQuotedLiteralKey` with FakeSource → FieldMapper → Console. The literal `$systemId` is asserted after decryption. Existing JSON single/batch coverage remains. Both new HOCON and existing JSON integration cases passed on the embedded two-member cluster. I removed the config-adapter guard and its test. I retained `ReadonlyConfig.toConfig()` with direct regression evidence: removing its guard made both JSON and HOCON REST requests return 500; the stack goes through `MultipleTableJobConfigParser.parseSource` → `MultiTableFailureHelper.mergeOptions` → `ReadonlyConfig.toConfig`. Restoring it made the tests pass. The English/Chinese REST docs now distinguish JSON path expressions from quoted HOCON literal keys. Local validation: 42 unit/REST tests and 2 selected integration tests passed; the 66-module dependency reactor completed `verify`. The PR description includes the integration command. The updated remote Build is still pending; these results are not a claim that CI is green or a commitment to a 3.0.1 backport. -- 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]
