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

Support engine operator #1043

Merged
merged 1 commit into from Nov 9, 2021
Merged

Conversation

wForget
Copy link
Member

@wForget wForget commented Oct 20, 2021

What is the purpose of the change

support engine operator. #1063

Brief change log

linkis-computation-client module

  • Define OperatorAction
  • Linkismanagerclient add executeOperate method to send OperatorAction to LinkisManager

linkis-application-manager module

  • Implement the DefaultEngineOperatorService in the LinkisManager to receive the operator request and execute it through the EngineNodeManager.
  • Add the executeOperation method in the EngineNodeManager to execute the EngineOperateRequest through the EngineNodPointer
  • The EngineNodPointer adds the executeOperation method to send the request to EngineConn

accessible-executor module

  • Implement the OperatorService class to receive EngineOperateRequest
  • Define the Operator interface, which can be implemented in the EngineConn plugins

Verifying this change

Not yet

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): (yes / no) no
  • Anything that affects deployment: (yes / no / don't know) no
  • The MGS(Microservice Governance Services), i.e., Spring Cloud Gateway, OpenFeign, Eureka.: (yes / no) no

Documentation

  • Does this pull request introduce a new feature? (yes / no) yes
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented) not documented

@wForget wForget changed the title support engine operator support engine operator (Incomplete) Oct 20, 2021
@wForget wForget changed the title support engine operator (Incomplete) Support engine operator Oct 25, 2021
<groupId>org.reflections</groupId>
</exclusion>
</exclusions>
<!-- <exclusions>-->
Copy link
Contributor

Choose a reason for hiding this comment

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

It is recommended to remove the comment code

Copy link
Contributor

@peacewong peacewong left a comment

Choose a reason for hiding this comment

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

LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants