hutiefang76 commented on PR #12443: URL: https://github.com/apache/seatunnel/pull/12443#issuecomment-5863236320
Thanks for checking the updated scope. My earlier reply was truncated; the intended rule is that keys accepted by the existing path parser retain its pre-existing normalization: `->` splits the path, syntactic quotes are removed, and surrounding unquoted whitespace is trimmed. Preserving those accepted keys byte-for-byte is outside this fix. Only keys rejected with `BadPath` are quoted and treated literally. I’ve made that explicit in the PR description. The REST notes are already included in both languages: [English](https://github.com/apache/seatunnel/blob/ddca53b7a591d1de12752156481e173a0fab3239/docs/en/engines/zeta/rest-api-v2.md#L921) and [Chinese](https://github.com/apache/seatunnel/blob/ddca53b7a591d1de12752156481e173a0fab3239/docs/zh/engines/zeta/rest-api-v2.md#L892). They cover reserved keys, `->`, quote/whitespace normalization, and batch submissions. The final diff touches `seatunnel-api`, `seatunnel-core-starter`, engine-server tests, engine E2E tests, and those two docs. I’ve changed the prefix to `[Fix][Core]` to reflect the shared conversion code. `MetadataProviderManager` and `AssertSink` remain explicitly outside this PR. -- 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]
