PowerJob
PowerJob copied to clipboard
feat: add queryInstanceInfoList and queryWorkflowInstanceInfoList
What is the purpose of the change
使得通过 OpenAPI 更好的进行任务查询 Make task query better through OpenAPI
Brief changelog
解决 #310 fix #310
Verifying this change
测试已完成 Test completed
详见: tech.powerjob.client.test.TestClient#testQueryInstanceInfoList and tech.powerjob.client.test.TestWorkflow#testQueryWorkflowInstanceInfoList see: tech.powerjob.client.test.TestClient#testQueryInstanceInfoList and tech.powerjob.client.test.TestWorkflow#testQueryWorkflowInstanceInfoList
哦吼,看了下 分布式任务调度 SchedulerX>API参考>GetJobInstanceList,发现忽略了一个问题,就是条数的限制。。。
该接口只返回最近10条运行实例列表。有空再改一下吧
坐等优化 @imhansai
哦吼,看了下 分布式任务调度 SchedulerX>API参考>GetJobInstanceList,发现忽略了一个问题,就是条数的限制。。。 该接口只返回最近10条运行实例列表。有空再改一下吧
坐等优化 @imhansai
看了下代码有几个疑惑:
- import 格式。现在没有样式文件,导致 import 部分风格不一样。希望能提供一下:)
- 看了 #158 这个issue。 代码位置: tech.powerjob.server.web.controller.OpenAPIController#queryInstance 这个看着像是提供了查询所有实例列表的能力。 但奇怪的是初始提交在 0912964ec3b975943ff16a060ba7af4d73e5480b 这个 commit id。 然后在 b044784e6a47085403f6f00eff8fd330777c785b 这个 commit id 仅仅去掉了 OhMyClient.java 的 queryInstanceInfo 方法。
- 代码位置还有排序、Repository查询时appId是否需要传递等等问题,不过这个可以在审查后再次修改:)
期待合并。