rangareddy commented on issue #16960: URL: https://github.com/apache/hudi/issues/16960#issuecomment-5391436401
This issue was reviewed as part of the JIRA-migrated backlog triage (HUDI-9292). **Findings: this cannot be worked in `apache/hudi` today - flagging it as a scope question.** There is **no Presto connector module in this repository**. Searching `master` for a `hudi-presto` module returns nothing. The only Presto artefacts here are: - `docker/hoodie/hadoop/prestobase/` and `docker/demo/presto-*.commands` (demo tooling) - `dependencies/hudi-presto-bundle.txt` (a bundle dependency list) - `rfc/rfc-44/presto-connector.png` (an RFC image) The connector itself lives in `prestodb/presto`, so adding filegroup reader based scan would have to happen there. **This is worth contrasting with Trino**, because the two are not symmetric any more: the Trino connector *was* migrated into this repo (`hudi-trino/`, commit `c3c936790727`, RFC-105 / #18837), which is why its sibling tickets in this batch are actionable here and these are not. So there is a decision to make rather than a defect to fix - either these move to `prestodb/presto`, or Hudi intends to bring the Presto connector in-repo the way it did Trino. I am not recommending closure either way, since that is a maintainer call. The Presto connector tickets in this batch are #16957, #16958, #16960 and #16969, and they share this same question. Keeping this open pending that decision. -- 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]
