Leo Q
Leo Q
This is good, why is there nobody reviewing this pr?
> > > I have no clue. I made this so long ago. Perhaps a maintainer can tell us what's wrong. > @asvoboda @bluekeyes You may set your own helm...
确实，代码在这里 https://github.com/hhyo/Archery/blob/6bb36f721481deaff1a29c29c26cd674d02620a3/sql/sql_workflow.py#L722
有冲突，解决下，另外mongo 的修改和pr无关，删掉或者另开一个pr吧
你的标题是多长? 我看一般情况下应该是够用的
api部分的测试好像少了点啊🤔，看能不能按CI的提示补一补
一般来说如果主机安全都不能保证，这里再怎么隐藏也是没有用的，进程只会存在一会，安全隐患应该是不高的，麻烦确认一下。如果在日志中没有出现的话，这个问题应该是不会去做的。
我说的就是包括宿主机在内的主机安全, 你说的这个方法是可以做的, 欢迎 pr https://github.com/hhyo/Archery/blob/master/sql/plugins/my2sql.py
https://github.com/hhyo/Archery/blob/6bb36f721481deaff1a29c29c26cd674d02620a3/sql/binlog.py#L141-L163 具体调用是在这里, plugin 加一个入口让调用方可以传递环境变量, 后面就可以加了
sqlserver 是我提的pr, 但我手边既没有 sqlserver, 公司也没有部署 archery, 麻烦你看看如何修复, 然后顺手提个 pr 吧