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

新增批量操作工单 #1531

Closed
wants to merge 2 commits into from
Closed

新增批量操作工单 #1531

wants to merge 2 commits into from

Conversation

cslingjun
Copy link
Contributor

@cslingjun cslingjun commented May 19, 2022

具体功能看图 #1469
middle_img_v2_6f98dcd6-0eb2-410d-afd1-ee57a308338g

@qinqi123456
Copy link

1.8.4 提交SQL已经支持实例配置的时候配上db_name。强烈建议支持“批量提交SQL”
(1)多租户场景需要支持“批量提交SQL”。
(2)批量时针对一个语句,希望可以插入到多个实例,多个schema。

@cslingjun
Copy link
Contributor Author

cslingjun commented May 23, 2022

1.8.4 提交SQL已经支持实例配置的时候配上db_name。强烈建议支持“批量提交SQL” (1)多租户场景需要支持“批量提交SQL”。 (2)批量时针对一个语句,希望可以插入到多个实例,多个schema。

这个功能我这边已经实现了 ,只是这个需求极少数朋友才会有。所以这块功能暂未提pr

@qinqi123456
Copy link

1.8.4 提交SQL已经支持实例配置的时候配上db_name。强烈建议支持“批量提交SQL” (1)多租户场景需要支持“批量提交SQL”。 (2)批量时针对一个语句,希望可以插入到多个实例,多个schema。

这个功能我这边已经实现了 ,只是这个需求极少数朋友才会有。所以这块功能暂未提pr

这个功能还是有必要的,易用性会提升一大截,尽快提pr吧,谢谢。

@codecov
Copy link

codecov bot commented May 23, 2022

Codecov Report

Merging #1531 (5fa259c) into master (4d96ac6) will decrease coverage by 0.87%.
The diff coverage is 4.57%.

@@            Coverage Diff             @@
##           master    #1531      +/-   ##
==========================================
- Coverage   76.84%   75.97%   -0.88%     
==========================================
  Files          91       91              
  Lines       14309    14484     +175     
==========================================
+ Hits        10996    11004       +8     
- Misses       3313     3480     +167     
Impacted Files Coverage Δ
sql/models.py 96.57% <ø> (ø)
sql/urls.py 100.00% <ø> (ø)
sql/sql_workflow.py 47.06% <4.57%> (-21.01%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4d96ac6...5fa259c. Read the comment docs.

@LeoQuote
Copy link
Collaborator

现在工单提交已经有接口了, 这个能否做成纯前端的功能? 因为后端这边做测试有点太麻烦了

@cslingjun
Copy link
Contributor Author

现在工单提交已经有接口了, 这个能否做成纯前端的功能? 因为后端这边做测试有点太麻烦了

这个代码我这边线上已经跑了快小一年了,这个后端处理会比较合适一点吧 ,我这边的需求有时候要处理1000多个工单,这样要反复调用1000多次接口 。不太合理感觉

Copy link
Collaborator

@LeoQuote LeoQuote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

代码复用不够, 这些检查的代码在单个工单的审核和执行中有类似的, 希望能复用一部分, 可以考虑直接把代码挪到 model 层, 作为 workflow 的一个 方法来做

@bluerfish
Copy link

1.8.4 提交SQL已经支持实例配置的时候配上db_name。强烈建议支持“批量提交SQL” (1)多租户场景需要支持“批量提交SQL”。 (2)批量时针对一个语句,希望可以插入到多个实例,多个schema。

这个功能我这边已经实现了 ,只是这个需求极少数朋友才会有。所以这块功能暂未提pr

这个对多租户和一些分库分表很有用啊,我们现在提相同语句的工单在不同实例要提很多次,也要审批很多次。能发出来么

@peng19832
Copy link
Contributor

现在工单提交已经有接口了, 这个能否做成纯前端的功能? 因为后端这边做测试有点太麻烦了

这个代码我这边线上已经跑了快小一年了,这个后端处理会比较合适一点吧 ,我这边的需求有时候要处理1000多个工单,这样要反复调用1000多次接口 。不太合理感觉

@cslingjun
大佬,这个功能,好像只考虑了审核人的批量审核相关操作,是不是应该加上提交人一侧的批量操作呢,比如:
同一个sql,需要同时提交到多个实例的多个数据库中:
这种场景,其实就是复用相同的代码和数据库结构,只是部署到不同子公司的不同部门中,当有迭代需求时,需要上线到所有这些数据库,这时候提交人,就很繁琐,需要提交好多次同一个sql

@cslingjun
Copy link
Contributor Author

现在工单提交已经有接口了, 这个能否做成纯前端的功能? 因为后端这边做测试有点太麻烦了

这个代码我这边线上已经跑了快小一年了,这个后端处理会比较合适一点吧 ,我这边的需求有时候要处理1000多个工单,这样要反复调用1000多次接口 。不太合理感觉

@cslingjun 大佬,这个功能,好像只考虑了审核人的批量审核相关操作,是不是应该加上提交人一侧的批量操作呢,比如: 同一个sql,需要同时提交到多个实例的多个数据库中: 这种场景,其实就是复用相同的代码和数据库结构,只是部署到不同子公司的不同部门中,当有迭代需求时,需要上线到所有这些数据库,这时候提交人,就很繁琐,需要提交好多次同一个sql

提交SQL按钮旁边是有个批量提交SQL工单按钮的 。你看我截图 。但是这块代码好像我没提上来

@Codecaver
Copy link

提交SQL按钮旁边是有个批量提交SQL工单按钮的 。你看我截图 。但是这块代码好像我没提上来

大佬好,可以指导一下怎么加这一块的代码吗,想要个批量提交SQL工单的功能,感谢感谢

@peng19832
Copy link
Contributor

提交SQL按钮旁边是有个批量提交SQL工单按钮的 。你看我截图 。但是这块代码好像我没提上来

大佬好,可以指导一下怎么加这一块的代码吗,想要个批量提交SQL工单的功能,感谢感谢

是的大佬,感谢

@cslingjun cslingjun closed this by deleting the head repository May 21, 2024
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

Successfully merging this pull request may close these issues.

None yet

6 participants