PowerJob icon indicating copy to clipboard operation
PowerJob copied to clipboard

feat: add queryInstanceInfoList and queryWorkflowInstanceInfoList

Open imhansai opened this issue 3 years ago • 4 comments

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

imhansai avatar Aug 18 '21 08:08 imhansai

哦吼,看了下 分布式任务调度 SchedulerX>API参考>GetJobInstanceList,发现忽略了一个问题,就是条数的限制。。。

该接口只返回最近10条运行实例列表。有空再改一下吧

imhansai avatar Sep 26 '21 07:09 imhansai

哦吼,看了下 分布式任务调度 SchedulerX>API参考>GetJobInstanceList,发现忽略了一个问题,就是条数的限制。。。

该接口只返回最近10条运行实例列表。有空再改一下吧

坐等优化 @imhansai

Echo009 avatar Sep 02 '22 08:09 Echo009

哦吼,看了下 分布式任务调度 SchedulerX>API参考>GetJobInstanceList,发现忽略了一个问题,就是条数的限制。。。 该接口只返回最近10条运行实例列表。有空再改一下吧

坐等优化 @imhansai

看了下代码有几个疑惑:

  1. import 格式。现在没有样式文件,导致 import 部分风格不一样。希望能提供一下:)
  2. 看了 #158 这个issue。 代码位置: tech.powerjob.server.web.controller.OpenAPIController#queryInstance 这个看着像是提供了查询所有实例列表的能力。 但奇怪的是初始提交在 0912964ec3b975943ff16a060ba7af4d73e5480b 这个 commit id。 然后在 b044784e6a47085403f6f00eff8fd330777c785b 这个 commit id 仅仅去掉了 OhMyClient.java 的 queryInstanceInfo 方法。
  3. 代码位置还有排序、Repository查询时appId是否需要传递等等问题,不过这个可以在审查后再次修改:)

imhansai avatar Sep 04 '22 07:09 imhansai

期待合并。

spance avatar Nov 11 '23 09:11 spance