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

Projection pushdown #676

Merged
merged 3 commits into from Jun 19, 2019

Conversation

4 participants
@martint
Copy link
Member

commented Apr 25, 2019

It supports basic column and row field dereference pushdown.

@cla-bot cla-bot bot added the cla-signed label Apr 25, 2019

@kokosing
Copy link
Member

left a comment

As I understand with this you would be able to remove getTableLayout entirely, am I right?

@martint martint force-pushed the martint:projection-pushdown branch from 41a7d00 to 1ae5191 May 1, 2019

@kokosing kokosing added WIP and removed WIP labels May 8, 2019

@martint martint referenced this pull request May 31, 2019

Open

Allow connectors to participate in query optimization #18

6 of 16 tasks complete

@martint martint force-pushed the martint:projection-pushdown branch 2 times, most recently from c981e39 to 1b15a21 Jun 10, 2019

@martint martint removed the WIP label Jun 11, 2019

@martint martint changed the title [WIP] Projection pushdown Projection pushdown Jun 11, 2019

@martint martint force-pushed the martint:projection-pushdown branch 7 times, most recently from 40d8eb2 to ac828f9 Jun 11, 2019

@martint martint force-pushed the martint:projection-pushdown branch from ac828f9 to acbf4c5 Jun 19, 2019

martint added some commits Jun 12, 2019

Fix default AstVisitor.visitRow implementation
Row is an Expression, so it should delegate to visitExpression
instead of jumping directly to visitNode
Add support for projection pushdown
Add a new applyProjection API to ConnectorMetadata. Currently, simple
column references and field dereference expressions are supported.

@martint martint force-pushed the martint:projection-pushdown branch from acbf4c5 to 5dccec2 Jun 19, 2019

@Praveen2112 Praveen2112 referenced this pull request Jun 19, 2019

Open

Remove table layouts from all connectors #781

10 of 15 tasks complete

@martint martint merged commit 1afbe98 into prestosql:master Jun 19, 2019

2 checks passed

Travis CI - Pull Request Build Passed
Details
verification/cla-signed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.