zhangshenghang commented on PR #12986: URL: https://github.com/apache/seatunnel/pull/12986#issuecomment-6682853064
Both open blockers are fixed in 03a9ec0f7: - Issue 3: the dialect-list rewrite lost in the dev merge is restored in all four pages (EN/ZH, source+sink) — `StarRocks` and `Inceptor` are removed as invalid dialect values, the real factory names `OpenGauss`/`Presto`/`DuckDB` are listed, and the verified `CockroachDB` entry from #12828 is kept. - Issue 1: the Kafka NATIVE note is corrected in EN and ZH — NATIVE inputs do not embed headers in the `value` byte array; headers are carried separately in the Kafka record headers, which is precisely why `kafka_headers_fields` is rejected with this format. Note on the blank-line nit: the exact spacing did not match on this head (the dev-side rows were reformatted), so that cosmetic tweak is left out; the dialect-list content fixes are the substance. Re: overlap with #13149 — this PR's dialect-table hunks and #13149's option-table hunks now touch disjoint regions, so whichever merges first should not conflict with the other. -- 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]
