Skip to content

Comments

[INLONG-5562][Manager] Add the workflow API in the manager client#5563

Merged
dockerzhang merged 5 commits intoapache:masterfrom
haifxu:feature-5562
Aug 19, 2022
Merged

[INLONG-5562][Manager] Add the workflow API in the manager client#5563
dockerzhang merged 5 commits intoapache:masterfrom
haifxu:feature-5562

Conversation

@haifxu
Copy link
Member

@haifxu haifxu commented Aug 16, 2022

Motivation

Supplement the API that the manager web exists but the client does not exist for manager client.

Modifications

Add the workflow API in the manager client

Verifying this change

  • This change added tests and can be verified as follows:
    • org.apache.inlong.manager.client.api.inner.WorkflowClientTest#testWorkflowStart
    • org.apache.inlong.manager.client.api.inner.WorkflowClientTest#testListProcess

@haifxu haifxu force-pushed the feature-5562 branch 2 times, most recently from 238219e to caedbf9 Compare August 17, 2022 02:09
@haifxu haifxu requested a review from leosanqing August 17, 2022 02:20
@haifxu haifxu force-pushed the feature-5562 branch 2 times, most recently from c2e2431 to 3963b24 Compare August 17, 2022 11:53
Copy link
Member

@healchow healchow left a comment

Choose a reason for hiding this comment

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

LGTM

@dockerzhang dockerzhang merged commit 88e95e8 into apache:master Aug 19, 2022
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.

[Feature][Manager] Supplement the workflow API in the manager client

4 participants