DanielLeens commented on issue #10914: URL: https://github.com/apache/seatunnel/issues/10914#issuecomment-4688435824
Thanks for linking the follow-up. One boundary is still important to keep explicit in this thread: PR #10964 is the routing foundation (PR-B0), but this issue is still the metadata-contract slice itself. The main review gate here has not changed: 1. freeze identity vs version/hash semantics 2. make the `document_id` / `chunk_id` vs `document_hash` / `chunk_hash` split explicit 3. show how those keys are projected through `MetadataTransform` 4. keep first-MVP non-goals explicit If PR #10990 is the implementation meant to carry PR-B itself, please use that as the main implementation link for this issue so reviewers do not conflate the routing foundation with the metadata contract. -- 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]
