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

fix(presto): Handle ROW data stored as string (#10456) #101

Merged
merged 1 commit into from
Jul 28, 2020

Conversation

betodealmeida
Copy link

Cherry-pick 4f67827 into the apache-0.36.0 branch. This fixes PRESTO_EXPAND_DATA.

* Handle ROW data stored as string

* Use destringify

* Fix mypy

* Fix mypy with cast

* Bypass pylint
@betodealmeida
Copy link
Author

👀 @khtruong @jinhyukchang

@codecov-commenter
Copy link

codecov-commenter commented Jul 28, 2020

Codecov Report

Merging #101 into apache-0.36.0 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##           apache-0.36.0     #101   +/-   ##
==============================================
  Coverage          59.03%   59.03%           
==============================================
  Files                379      379           
  Lines              12181    12181           
  Branches            3010     3010           
==============================================
  Hits                7191     7191           
  Misses              4809     4809           
  Partials             181      181           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2cd8ca9...be1c489. Read the comment docs.

@betodealmeida
Copy link
Author

🚀

@betodealmeida betodealmeida merged commit 1e9319d into apache-0.36.0 Jul 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants