yanghua commented on pull request #2868:
URL: https://github.com/apache/hudi/pull/2868#issuecomment-828977301


   > > > Fine, let's keep them for a time.
   > > 
   > > 
   > > I do not object to deleting the legacy code. I just suggest we'd better 
add the Kafka connector into the bundle. And let it be a completed function. 
Then, we can delete the legacy code and use the new Flink streamer.
   > 
   > I agree the Flink streamer should bundle with a kafka jar, but the 
hudi-flink-bundle should not include kafka, because many people include the 
bundle jar in the pom directly, we should keep the jar as small as possible to 
avoid unnecessary jar conflicts.
   > 
   > Maybe we should move the streamer out of the hudi-flink module into the 
hudi-utilities module.
   
   +1, it's what I mean.


-- 
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:
[email protected]


Reply via email to