Skip to content

Conversation

@wyb
Copy link
Contributor

@wyb wyb commented May 28, 2020

  1. Fe checks the status of etl job regularly
    1.1 If status is RUNNING, update etl job progress
    1.2 If status is CANCELLED, cancel load job
    1.3 If status is FINISHED, get the etl output file paths, update job state to LOADING and log job update info

  2. Fe sends PushTask to Be and commits transaction after all push tasks execute successfully

#3433

@imay imay added api-review Categorizes an issue or PR as actively needing an API review. area/load Issues or PRs related to all kinds of load kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API kind/feature Categorizes issue or PR as related to a new feature. labels May 29, 2020
@wyb wyb changed the title [Spark load] Fe process etl and loading state job [Spark load][Fe 1/5] Fe process etl and loading state job May 30, 2020
@wyb wyb changed the title [Spark load][Fe 1/5] Fe process etl and loading state job [Spark load][Fe 5/5] Fe process etl and loading state job May 30, 2020
@wyb wyb changed the title [Spark load][Fe 5/5] Fe process etl and loading state job [Spark load][Fe 6/6] Fe process etl and loading state job Jun 10, 2020
@wyb wyb force-pushed the spark_load_fe_etl_loading branch from d7c944a to b0aaf1e Compare June 19, 2020 10:42
@wyb wyb force-pushed the spark_load_fe_etl_loading branch from b0aaf1e to 1da4965 Compare June 19, 2020 11:07
morningman
morningman previously approved these changes Jun 19, 2020
Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

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

LGTM

@morningman
Copy link
Contributor

The compile error of broker has been fix in #3913 .
So I just approve this PR.

@morningman morningman merged commit a63fa88 into apache:master Jun 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api-review Categorizes an issue or PR as actively needing an API review. area/load Issues or PRs related to all kinds of load kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API kind/feature Categorizes issue or PR as related to a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants