Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
suxg001 opened this issue Dec 20, 2019 · 3 comments
Open

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

suxg001 opened this issue Dec 20, 2019 · 3 comments

Comments

@suxg001
Copy link

suxg001 commented Dec 20, 2019

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

@shzlw
Copy link
Owner

shzlw commented Jan 4, 2020

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

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

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

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

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

@suxg001
Copy link
Author

suxg001 commented Jan 4, 2020

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

@suxg001
Copy link
Author

suxg001 commented Jan 4, 2020

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants