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

   Thank you for the sustained effort behind this contribution. Adding PDF 
parsing for the file connector is a meaningful capability for RAG and 
document-processing users, and we really appreciate the care put into the 
implementation, regression PDF, tests, and the English and Chinese 
documentation.
   
   I rechecked the conflict-free head after syncing the latest `dev`. The two 
HDFS documentation conflicts now preserve both sides correctly: the upstream 
multi-table `tables_configs` documentation remains intact, and the new PDF 
format and parsing behavior are documented in both languages. The focused PDF 
regression suite passed all 7 tests locally, and formatting also passed.
   
   From Daniel’s side, I do not see a remaining source-level blocker on this 
updated head. The fresh Build run is now the remaining automated gate before 
merge.
   
   Thank you again for contributing this feature and for continuing to work 
with the community through several rounds of updates. That persistence is 
valuable, and we are very glad to have your participation in Apache SeaTunnel.


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