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

visualize canvas pipeline #23185

Closed
wants to merge 8 commits into from

Conversation

ppisljar
Copy link
Member

@ppisljar ppisljar commented Sep 14, 2018

using canvas pipeline to fetch data inside visualize

resolves #19813

QA: nothing should change functionally

@ppisljar ppisljar added the WIP Work in progress label Sep 14, 2018
@elasticmachine
Copy link
Contributor

💔 Build Failed

@trevan
Copy link
Contributor

trevan commented Sep 14, 2018

@ppisljar, doesn't this break OSS Kibana? Since OSS Kibana doesn't have x-pack, it won't have canvas so some of those imports will fail.

@ppisljar ppisljar added Feature:ExpressionLanguage Interpreter expression language (aka canvas pipeline) Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.0.0 labels Sep 16, 2018
@ppisljar
Copy link
Member Author

ppisljar commented Sep 17, 2018

@trevan this is WIP, still a long road ahead, we will be moving the parts necessary to OSS by then.

@ppisljar ppisljar force-pushed the visualize_canvas_pipeline branch from 8f41b90 to 569bf24 Compare September 17, 2018 09:36
@ppisljar ppisljar requested a review from timroes September 17, 2018 09:37
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@ppisljar ppisljar force-pushed the visualize_canvas_pipeline branch from 83bdb5c to 7af62da Compare September 17, 2018 17:51
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@ppisljar ppisljar mentioned this pull request Nov 7, 2018
1 task
@ppisljar ppisljar force-pushed the visualize_canvas_pipeline branch from dc1e225 to 3a2e8e0 Compare November 7, 2018 15:37
@elasticmachine
Copy link
Contributor

💔 Build Failed

@ppisljar ppisljar force-pushed the visualize_canvas_pipeline branch from 3a2e8e0 to e2f900f Compare November 7, 2018 16:09
@elasticmachine
Copy link
Contributor

💔 Build Failed

@timroes timroes mentioned this pull request Nov 8, 2018
5 tasks
@ppisljar ppisljar mentioned this pull request Nov 15, 2018
2 tasks
@ppisljar ppisljar force-pushed the visualize_canvas_pipeline branch from e2f900f to a2f09ea Compare November 20, 2018 08:59
@ppisljar ppisljar requested a review from lukeelmers November 20, 2018 09:00
@elasticmachine
Copy link
Contributor

💔 Build Failed

@ppisljar ppisljar force-pushed the visualize_canvas_pipeline branch from a2f09ea to 5062baf Compare November 20, 2018 16:46
@elasticmachine
Copy link
Contributor

💔 Build Failed

@ppisljar ppisljar force-pushed the visualize_canvas_pipeline branch from 5062baf to e2bc5ba Compare November 21, 2018 08:49
@ppisljar ppisljar force-pushed the visualize_canvas_pipeline branch from e2bc5ba to 62e7126 Compare November 21, 2018 09:11
@ppisljar
Copy link
Member Author

closing in favor of #25996

@ppisljar ppisljar closed this Nov 21, 2018
@elasticmachine
Copy link
Contributor

💔 Build Failed

@ppisljar ppisljar mentioned this pull request Nov 22, 2018
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:ExpressionLanguage Interpreter expression language (aka canvas pipeline) Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.0.0 WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use the Canvas rendering pipeline/expressions for all visualizations
3 participants