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

[ML] Data frames: Updated progress reporting. #39920

Merged

Conversation

walterra
Copy link
Contributor

@walterra walterra commented Jun 28, 2019

Summary

  • Adds a column mode to the data frame transforms list to indicate if the transform is batch or continuous.
  • For continuous data frames an animated horizontal loading indicator is shown in the Progress column instead of a progress bar with percentage.
  • In expanded rows, for Job details the section checkpointing was added.
  • In the transform wizard's creation step, for continuous transforms the progress bar is not shown. In a follow up we could add more useful stats there, like what's shown in the transform list's job details.

image

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

Tested (including IE11) and LGTM.
Would be nice if the timestamps in the expanded row of the transforms list could be converted to human readable formatted date / times, bont one for a follow-up I guess.

@walterra
Copy link
Contributor Author

walterra commented Jul 1, 2019

Created #40000 for the timestamp formatting 😛

Copy link
Member

@jgowdyelastic jgowdyelastic left a comment

Choose a reason for hiding this comment

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

LGTM

@walterra walterra merged commit 4426094 into elastic:master Jul 1, 2019
@walterra walterra deleted the ml-data-frames-continuous-progress branch July 1, 2019 11:59
walterra added a commit to walterra/kibana that referenced this pull request Jul 1, 2019
- Adds a column mode to the data frame transforms list to indicate if the transform is batch or continuous.
- For continuous data frames an animated horizontal loading indicator is shown in the Progress column instead of a progress bar with percentage.
- In expanded rows, for Job details the section checkpointing was added.
- In the transform wizard's creation step, for continuous transforms the progress bar is not shown. In a follow up we could add more useful stats there, like what's shown in the transform list's job details.
walterra added a commit that referenced this pull request Jul 1, 2019
- Adds a column mode to the data frame transforms list to indicate if the transform is batch or continuous.
- For continuous data frames an animated horizontal loading indicator is shown in the Progress column instead of a progress bar with percentage.
- In expanded rows, for Job details the section checkpointing was added.
- In the transform wizard's creation step, for continuous transforms the progress bar is not shown. In a follow up we could add more useful stats there, like what's shown in the transform list's job details.
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.

4 participants