-
Notifications
You must be signed in to change notification settings - Fork 109
chore: relocate workflow-compiling-service #3868
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
chore: relocate workflow-compiling-service #3868
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR moves the core/workflow-compiling-service directory to the root level as part of a repository reorganization plan. The changes update file paths and build configurations to reflect the new directory structure.
- Updated file paths in test files to reference the new location
- Modified build configurations to work with the relocated service
- Updated GitHub Actions workflow to reflect the new structure
Reviewed Changes
Copilot reviewed 7 out of 45 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| workflow-compiling-service/src/test/scala/org/apache/texera/service/resource/WorkflowCompilationResourceSpec.scala | Updated CSV file path to reflect new directory structure |
| core/workflow-core/src/test/scala/org/apache/amber/storage/FileResolverSpec.scala | Updated CSV file path to include core/ prefix |
| core/dao/src/test/scala/org/apache/texera/dao/MockTexeraDB.scala | Updated DDL script path to include core/ prefix |
| core/dao/src/main/resources/jooq-conf.xml | Updated directory path for generated classes |
| core/amber/build.sbt | Changed project name from "texera" to "amber" |
| .github/workflows/github-action-build.yml | Removed cd core && from sbt commands |
| .github/labeler.yml | Added new service label configuration |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
c333bf9 to
a839d7f
Compare
Signed-off-by: Yicong Huang <17627829+Yicong-Huang@users.noreply.github.com>
76c4447 to
537494b
Compare
I can’t build it on my machine; it might be just my local issue. @bobbai00, could you please test it on your machine? If it works for you, feel free to approve and merge the PR.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM

According to the reorganization plan #3846. This PR moves
core/workflow-compiling-serviceout to root.resolves #3860