Skip to content

Conversation

@Yicong-Huang
Copy link
Contributor

@Yicong-Huang Yicong-Huang commented Oct 15, 2024

This PR adds a new sub-project in sbt called workflow-core, under micro-services.
It contains a duplicated codebase of workflow core dependencies, including:

  • Tuple, Attribute, Schema;
  • OperatorExecutor;
  • PhysicalOp, PhysicalPlan;
  • Identities;
  • and other utility functions.

Migration plan:

After exporting workflow-core as a local dependency, we can build workflow-compiling-service and workflow-execution-service based on it.

@Yicong-Huang Yicong-Huang requested a review from bobbai00 October 15, 2024 20:21
@Yicong-Huang Yicong-Huang self-assigned this Oct 15, 2024
@Yicong-Huang Yicong-Huang added the refactor Refactor the code label Oct 15, 2024
Copy link
Contributor

@bobbai00 bobbai00 left a comment

Choose a reason for hiding this comment

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

Left one minor. LGTM!

@bobbai00
Copy link
Contributor

Can other micro services like workflow-compiling-service import workflow-core as dependencies after this PR?

@Yicong-Huang
Copy link
Contributor Author

Can other micro services like workflow-compiling-service import workflow-core as dependencies after this PR?

That is the idea. I sent you a POC branch as an example for using workflow-core as dependency.

@Yicong-Huang Yicong-Huang merged commit 78b0a49 into master Oct 16, 2024
@Yicong-Huang Yicong-Huang deleted the yicong-create-workflow-core branch October 16, 2024 01:17
PurelyBlank pushed a commit that referenced this pull request Dec 4, 2024
…2942)

This PR adds a new sub-project in sbt called `workflow-core`, under
micro-services.
It contains a duplicated codebase of workflow core dependencies,
including:
- `Tuple`, `Attribute`, `Schema`;
- `OperatorExecutor`;
- `PhysicalOp`, `PhysicalPlan`;
- Identities;
- and other utility functions.

### Migration plan:
After exporting workflow-core as a local dependency, we can build
workflow-compiling-service and workflow-execution-service based on it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Refactor the code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants