添加 pgsql 上线功能 #334
添加 pgsql 上线功能 #334
Conversation
Codecov Report
@@ Coverage Diff @@
## master #334 +/- ##
==========================================
- Coverage 85.29% 85.09% -0.21%
==========================================
Files 67 67
Lines 8822 8955 +133
==========================================
+ Hits 7525 7620 +95
- Misses 1297 1335 +38
Continue to review full report at Codecov.
|
|
你已经很久没有回复这个issue了，如果没有进一步的信息的话， 会作为不活跃issue关闭， 感谢你对本项目的贡献。 |
sql/engines/pgsql.py
Outdated
| def get_connection(self, db_name): | ||
| #只有获取数据库名字可以不带dbname参数 | ||
| if not db_name and traceback.extract_stack()[-3][2] != "get_all_databases": | ||
| raise ValueError('db_name未填写,请检查参数') |
There was a problem hiding this comment.
可以考虑在query 函数中检查参数, 这里还要看堆栈感觉挺蛋疼
There was a problem hiding this comment.
这里怎么写 已经考虑到 只有 not db_name 才会 去 查看 traceback.extract_stack()[-3][2]
There was a problem hiding this comment.
新做一个 _query 方法, 这个方法不检查 db_name, schema_name 等, 就只是查询, query 方法调用_query 方法, 但是在调用前检查 db_name, schema_name
get_all_database 的时候直接使用 _query 方法, 不检查参数
|
@yzypals 聚聚, 没问题的话按上面的评论改一下吧, 合了好release 一波啊 |
|
嗯 好 的 主要最近忙着搞elk |
|
又关了, 不提了? |
|
这是关了还是忙完别的再继续做哇？大佬们，不要放弃这个啊 |
|
挂这么久 我都不好意思了 ， 我 今天 去修改下 |
优化pgsql创建连接逻辑
暂时解决pgsql无法查看表结构问题
暂时解决query 参数传递问题
|
单元测试没过, 有空看一下. |
|
@hhyo 问题不大，可以合了 |
优化pgsql创建连接逻辑
暂时解决pgsql无法查看表结构问题
暂时解决query 参数传递问题
关联issue : #204