Thanks for the feedback, raised a PR[1] to implement option 4 [1] https://github.com/apache/incubator-kyuubi/pull/1147
Thanks, Cheng Pan On Fri, Sep 24, 2021 at 9:42 AM wangsheng <[email protected]> wrote: > +1, for option 4 > > > 2021年9月22日 下午2:52，Cheng Pan <[email protected]> 写道： > > > > Hi developers, > > > > Currently, we have a module named `kyuubi-hive-jdbc`, but the classes' > > package name lacks `hive`, for instance, > > `org.apache.kyuubi.jdbc.KyuubiDriver`, it's a bit inconsistent. > > > > I propose 3 change options to make it consistent. > > > > 1. remame `kyuubi-hive-jdbc` to `kyubbi-jdbc` > > 2. change pacakge `org.apache.kyuubi.jdbc` to > `org.apache.kyuubi.jdbc.hive` > > 3. change pacakge `org.apache.kyuubi.jdbc` to > `org.apache.kyuubi.hive.jdbc` > > > > I'd like to hear what other people think. > > > > Thanks, > > Cheng Pan > > > >
