You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2021/01/06 23:30:16 UTC

[GitHub] [hudi] umehrot2 edited a comment on pull request #2410: [HUDI-1510] Move HoodieEngineContext and its dependencies to hudi-common

umehrot2 edited a comment on pull request #2410:
URL: https://github.com/apache/hudi/pull/2410#issuecomment-755777542


   @yanghua Thank you for the feedback and bringing up a good discussion. Me and Vinoth had a quick call on this, and we still feel that option A of moving `HoodieEngineContext` over to `hudi-common` would be more reasonable. By moving this to `hudi-common` and having like a `HoodieMREngineContext` or `HoodieQueryEngineContext`, different query engines can potentially be able to use the parallelization optimizations. That is why we would want to move just the interface over to `hudi-common`. The query side engine contexts can sit in `hudi-common`, while the writer side engine contexts will continue to be in `hudi-client`. Please let us know if you have concerns with this.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org