velox
velox copied to clipboard
facebookincubator
Support reading plain encoded INT96 timestamp from Parquet file
Deploy Preview for meta-velox canceled.
| Name | Link |
|---|---|
| Latest commit | 316bf01134b65160061a6fca977387372a9ab45a |
| Latest deploy log | https://app.netlify.com/sites/meta-velox/deploys/673164d2db5126000895f227 |
![netlify[bot] avatar](https://avatars.githubusercontent.com/in/13473?v=4 "Published by a pub.dev verified publisher"){kind=small}
Hi @mskapilks, since Gluten customer has been asking for this feature, I created this PR to add support for reading plain encoded INT96 timestamp with your commit kept. If you would like to continue this work in your PR, please let us know. Thanks!
Hi @mskapilks, since Gluten customer has been asking for this feature, I created this PR to add support for reading plain encoded INT96 timestamp with your commit kept. If you would like to continue this work in your PR, please let us know. Thanks!
Thanks for the PR You can continue. I am still looking into INT64 pr
@kevinwilfong has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@kevinwilfong This PR might need to be re-imported before merging. Thanks for your help.
@kevinwilfong has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@kevinwilfong merged this pull request in facebookincubator/velox@da39954c22f1dda1d4b78dae078fb1307fc2508f.
Conbench analyzed the 1 benchmark run on commit da39954c.
There were no benchmark performance regressions. 🎉
The full Conbench report has more details.
![conbench-facebook[bot] avatar](https://avatars.githubusercontent.com/in/241235?v=4 "Published by a pub.dev verified publisher"){kind=small}