DanielLeens commented on PR #10779:
URL: https://github.com/apache/seatunnel/pull/10779#issuecomment-5165359302

   Thanks for clarifying that intention.
   
   In that case, I would still avoid merging the parser-only path first on the 
current head. The blocker I raised earlier remains the same: this PR still 
needs to show the real pgoutput/Debezium runtime input path that feeds 
`PostgresSchemaChangeResolver`, or otherwise narrow the scope so we do not 
merge parsing complexity that is not yet wired into the actual PostgreSQL CDC 
execution path.
   
   Once that is reflected in code, I am happy to re-review the updated head.


-- 
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]

Reply via email to