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

Load tables dynamically in Elasticsearch #1639

Merged
merged 3 commits into from Oct 4, 2019

Conversation

@martint
Copy link
Member

commented Oct 2, 2019

It's no longer necessary to have a static configuration to declare which tables are available and their schema. They are now derived from the metadata exposed by ES.

The following data types are now supported: tinyint, smallint, integer, real, binary, timestamp. Additionally, the _id, _score and _source (hidden) columns are also available for querying.

@cla-bot cla-bot bot added the cla-signed label Oct 2, 2019
@martint martint requested review from dain and zhenxiao Oct 2, 2019
@martint martint force-pushed the martint:elastic-dynamic branch 2 times, most recently from 7dcd4a5 to dc66d7f Oct 2, 2019
martint added 3 commits Sep 28, 2019
Also adds support for date, binary, tinyint, smallint and real data types
The current defaults are too aggressive. Use something more reasonable.
@martint martint force-pushed the martint:elastic-dynamic branch from 360603e to c43d1d0 Oct 4, 2019
@martint martint merged commit c43d1d0 into prestosql:master Oct 4, 2019
1 check passed
1 check passed
verification/cla-signed
Details
@martint martint deleted the martint:elastic-dynamic branch Oct 4, 2019
@martint martint referenced this pull request Oct 4, 2019
6 of 6 tasks complete
@martint martint added this to the 320 milestone Oct 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.