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

[Feature][Zeta][REST-API]add get running job info by jobId #4140

Merged
merged 30 commits into from
Mar 22, 2023

Conversation

liugddx
Copy link
Member

@liugddx liugddx commented Feb 15, 2023

Purpose of this pull request

Check list

@liugddx liugddx requested a review from Hisoka-X March 13, 2023 03:11
@Hisoka-X
Copy link
Member

Please add e2e test case for this change.

@TyrantLucifer
Copy link
Member

Please add e2e test case for this change.

Yup, add e2e test case is better.

@liugddx
Copy link
Member Author

liugddx commented Mar 13, 2023

Please add e2e test case for this change.

Yup, add e2e test case is better.

Ok, I will provide e2e.

@liugddx liugddx changed the title [Feature][Zeta][REST-API]add get running job info by jobId [WIP][Feature][Zeta][REST-API]add get running job info by jobId Mar 13, 2023
@liugddx liugddx marked this pull request as draft March 13, 2023 15:09
@liugddx liugddx changed the title [WIP][Feature][Zeta][REST-API]add get running job info by jobId [Feature][Zeta][REST-API]add get running job info by jobId Mar 15, 2023
@liugddx liugddx marked this pull request as ready for review March 15, 2023 14:44
@liugddx liugddx merged commit 4e52eba into apache:dev Mar 22, 2023
@liugddx liugddx deleted the feature-jobinfo branch March 22, 2023 02:10
MonsterChenzhuo pushed a commit to MonsterChenzhuo/incubator-seatunnel that referenced this pull request Apr 19, 2023
* add get running job info by jobId

* add release note

* fix ci error

* fix ci error

* Use constant string to manage url path;

* add e2e test case

* add e2e case

* add e2e case

* add e2e case

* add e2e test case

* add e2e test case

* add e2e case

* add e2e case

* change method name

* revert

* get port for rest api

* add e2e test

* add e2e test

* Update rest-api.md

* Update rest-api.md

* Update docs/en/seatunnel-engine/rest-api.md

* code style

* code style

* code style

* code style

* code style

* close zeta engine after all test case done.

---------

Co-authored-by: Hisoka <[email protected]>
ic4y pushed a commit to ic4y/incubator-seatunnel that referenced this pull request May 22, 2023
* add get running job info by jobId

* add release note

* fix ci error

* fix ci error

* Use constant string to manage url path;

* add e2e test case

* add e2e case

* add e2e case

* add e2e case

* add e2e test case

* add e2e test case

* add e2e case

* add e2e case

* change method name

* revert

* get port for rest api

* add e2e test

* add e2e test

* Update rest-api.md

* Update rest-api.md

* Update docs/en/seatunnel-engine/rest-api.md

* code style

* code style

* code style

* code style

* code style

* close zeta engine after all test case done.

---------

Co-authored-by: Hisoka <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants