bowenli86 edited a comment on issue #8809: [FLIN-12663]Implement HiveTableSource to read Hive tables URL: https://github.com/apache/flink/pull/8809#issuecomment-505146914 @KurtYoung regarding your comment on the package name, what's your suggestion on a proper name? It's been brought by @zjffdu too before. I think @zjuwangg named it this way because most connector packages are named as `org.apache.flink.streaming.connectors.xxx` and he is just following the convention. However, as we are forwarding to streaming-batch unification, we probably don't need "streaming/batch" in the package names any more, coz, like file source/sink, hive source/sink can (doesn't mean we necessarily will) be made as streaming in the future. I'm thinking of just `org.apache.flink.connectors.hive`. What do you think? @zjuwangg can you please create a JIRA ticket to track this discussion? We probably need to finalize the package name before releasing 1.9 (not necessarily in this PR), otherwise it's hard to change. cc @xuefuz @lirui-apache
---------------------------------------------------------------- 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] With regards, Apache Git Services
