Skip to content

[Feature][Zeta][REST-API] Add REST API Get all completed jobs informa…#4791

Closed
MonsterChenzhuo wants to merge 2 commits intoapache:devfrom
MonsterChenzhuo:api
Closed

[Feature][Zeta][REST-API] Add REST API Get all completed jobs informa…#4791
MonsterChenzhuo wants to merge 2 commits intoapache:devfrom
MonsterChenzhuo:api

Conversation

@MonsterChenzhuo
Copy link
Contributor

@MonsterChenzhuo MonsterChenzhuo commented May 19, 2023

Purpose of this pull request

close:#4145

Check list

@MonsterChenzhuo
Copy link
Contributor Author

图片

@MonsterChenzhuo
Copy link
Contributor Author

MonsterChenzhuo commented May 19, 2023

@EricJoy2048 @liugddx @hailin0 PTAL, thanks.

@liugddx
Copy link
Member

liugddx commented May 20, 2023

Can we get the statistics of the job? The amount of data etc.

@MonsterChenzhuo
Copy link
Contributor Author

图片 I added more task information

@liugddx
Copy link
Member

liugddx commented May 20, 2023

图片 I added more task information

The structure of the job is job->pipeline->taskGroup->task, when a job has multiple pipelines, the user may want to divide the pipelines for statistics,Can we count according to the nested structure of job->pipelines->taskGroup?

{ "jobId":"", "pipelines":[ { "pipelineId":"", "sourcet":[ ], "Sink":[] } ] }

@EricJoy2048 @hailin0 @ic4y @Hisoka-X any suggestion?

Copy link
Member

@TyrantLucifer TyrantLucifer left a comment

Choose a reason for hiding this comment

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

add UT/IT test case

@MonsterChenzhuo
Copy link
Contributor Author

add UT/IT test case

seatunnel-e2e/seatunnel-engine-e2e/connector-seatunnel-e2e-base/src/test/java/org/apache/seatunnel/engine/e2e/RestApiIT.java There are corresponding additions below.

@liugddx
Copy link
Member

liugddx commented Jun 27, 2023

@MonsterChenzhuo Can it be output according to this structure?

@MonsterChenzhuo
Copy link
Contributor Author

@MonsterChenzhuo Can it be output according to this structure?
Oh, okay.

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.

3 participants