Skip to content

[7.x] Table visualization renderer (#79455)#80121

Merged
sulemanof merged 1 commit intoelastic:7.xfrom
sulemanof:backport/7.x/pr-79455
Oct 12, 2020
Merged

[7.x] Table visualization renderer (#79455)#80121
sulemanof merged 1 commit intoelastic:7.xfrom
sulemanof:backport/7.x/pr-79455

Conversation

@sulemanof
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* Implement toExpressionAst fn

* Implement table vis renderer, move legacy codebase

* Update legacy paths

* Update types

* Fix filtering

* Update styles

* Remove legacy tests

* Update docs

* Update tests

* Fix tests

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
# Conflicts:
#	docs/development/plugins/expressions/public/kibana-plugin-plugins-expressions-public.iinterpreterrenderhandlers.md
#	docs/development/plugins/expressions/server/kibana-plugin-plugins-expressions-server.iinterpreterrenderhandlers.md
#	src/plugins/expressions/public/public.api.md
#	src/plugins/expressions/server/server.api.md
@sulemanof sulemanof added the backport This PR is a backport of another PR label Oct 12, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

@kbn/optimizer bundle module count

id before after diff
visTypeTable 42 44 +2

async chunks size

id before after diff
visTypeTable 6.7KB 70.8KB +64.1KB

distributable file count

id before after diff
default 48761 48767 +6
oss 29468 29474 +6

page load bundle size

id before after diff
visTypeTable 78.2KB 20.2KB -58.0KB
visualizations 273.6KB 272.9KB -753.0B
total -58.8KB

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@sulemanof sulemanof merged commit c70fdaa into elastic:7.x Oct 12, 2020
@sulemanof sulemanof deleted the backport/7.x/pr-79455 branch October 12, 2020 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants