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

[pixels-presto] upgrade Presto spi version. #132

Closed
bianhq opened this issue Oct 19, 2021 · 2 comments
Closed

[pixels-presto] upgrade Presto spi version. #132

bianhq opened this issue Oct 19, 2021 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@bianhq
Copy link
Contributor

bianhq commented Oct 19, 2021

Upgrade the version of Presto spi, to explore new features in Presto.

@bianhq bianhq added the enhancement New feature or request label Oct 19, 2021
@bianhq bianhq self-assigned this Oct 19, 2021
@bianhq
Copy link
Contributor Author

bianhq commented Oct 24, 2021

In this issue, we only upgrade pixels connector to a Presto version between 0.208 and 0.215.

We plan to re-develop the connector for the latest Trino and/or PrestoDB later.

bianhq added a commit that referenced this issue Oct 27, 2021
…138)

Upgrade the Presto version we support from 0.192 to 0.215, so that we can benefit from the GC and performance inprovement introduced by 0.215.

During the upgrading, we also found and fixed bugs in VarcharArrayBlock, PixelsPageSource, and PixelsRecordReaderImpl. The implementation of ColumnVectors are also refined.
@bianhq bianhq added this to the GC Optimization milestone Oct 27, 2021
@bianhq
Copy link
Contributor Author

bianhq commented Oct 27, 2021

Resolved, the upgrading helps reduce GC overhead, so we connect it to the GC Optimization milestone.

@bianhq bianhq closed this as completed Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant