search

shzlw / poli

An easy-to-use BI server built for SQL lovers. Power data analysis in SQL and gain faster business insights.
https://shzlw.github.io/poli
MIT License
1.97k stars 330 forks source link

pr文件第三方jar包怎么合并 #57

Open suxg001 opened 4 years ago

suxg001 commented 4 years ago

我修改了一版源码，其中涉及到一个私有jar包，已经pr，请看下是否合并

shzlw commented 4 years ago

谢谢PR，第三方的包如果是源码里要用的，那只能改源码了。

JDBC jar包没有被加到pom里的原因是

  1. jdbc driver有license的问题
  2. 我不知道用户会用什么数据库，不可能把所有的数据库driver都加进来。所以做成driver放在jdbc-drivers下就可以在运行时加载了。用户自己选择。

PR里你还修改了源码中修改了logback的配置，这个其实我也是想让有需求的用户，可以在运行时外部加载自己需要的配置，这样就可以覆盖源码里的了。--logging.config=logback.xml

最后你commit的application.properties里好像暴露数据库的连接信息了。

suxg001 commented 4 years ago

我代码里实现了默认为全部查询 ，（and or） 这些语句用# 来分割sql语句，这样默认就查全部，当and 后的值有内容时候，即可走条件查询。可参考一下，其实咱们这个属于工具类型，可以走产品形态，就是认为使用者只关心业务sql即可，其他我们都帮他们实现

suxg001 commented 4 years ago

日志包我用了阿里巴巴内部未开源的包，实现了多线程ID并行切面日志。jar包在doc 源里有jar文件，后续我把这个打个子工程提交一下。 对了还有一个问题就是 图形报表我看查询条件都是以value的内容来直接查询的，建议改为id查询，因为数据库一般设计为多表关联，直接用pid查询是不是会更方便一些。 当前代码里这种查询在相同值的情况下有多个值会出现，并且查询性能上也无法走索引，请参考，谢谢