Skip to content

[Pinot Connector] Support querying Pinot JSON type#17323

Merged
highker merged 1 commit intoprestodb:masterfrom
xiangfu0:fixing-json-type-conversion-in-segment-page-source
Feb 18, 2022
Merged

[Pinot Connector] Support querying Pinot JSON type#17323
highker merged 1 commit intoprestodb:masterfrom
xiangfu0:fixing-json-type-conversion-in-segment-page-source

Conversation

@xiangfu0
Copy link
Copy Markdown
Contributor

@xiangfu0 xiangfu0 commented Feb 18, 2022

Test plan - (Please fill in how you tested your changes)

Local cluster test, based on top of Pinot JsonIndexQuickStart.

Before:

presto:default> SELECT json_extract(repo, '$.name'), count(*) from githubEvents  group by 1;
Query 20220218_051408_00009_tar98 failed: Internal error

After

presto:default> SELECT json_extract(repo, '$.name'), count(*) from githubEvents group by 1;
                                     _col0                                      | _col1
--------------------------------------------------------------------------------+-------
 "g0tmi1k/exploit-database"                                                     |     2
 "ctfs/write-ups-2014"                                                          |     1
 "miyabishi/waltz_editor"                                                       |     1
 "th174/LegitiFact-source-checker-bot"                                          |     1
 "g0v-data/mirror-minutely"                                                     |    22
 "lukemassa/jclubtakeaways"                                                     |     2
 "treckstar/yolo-octo-hipster"                                                  |     1
...

Fill in the release notes towards the bottom of the PR description.
See Release Notes Guidelines for details.

== RELEASE NOTES ==

Pinot Changes
* Support querying Pinot JSON type

@xiangfu0 xiangfu0 changed the title Support json type in PinotSegmentPageSource [Pinot Connector] Support querying Pinot JSON type Feb 18, 2022
@xiangfu0
Copy link
Copy Markdown
Contributor Author

@highker

@highker highker merged commit 6b5da9a into prestodb:master Feb 18, 2022
@xiangfu0 xiangfu0 deleted the fixing-json-type-conversion-in-segment-page-source branch February 18, 2022 19:28
@varungajjala varungajjala mentioned this pull request Mar 22, 2022
9 tasks
@asjadsyed asjadsyed mentioned this pull request Mar 23, 2022
9 tasks
@asjadsyed asjadsyed mentioned this pull request Apr 1, 2022
8 tasks
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.

2 participants