Skip to content

Add support for JSON type in Pinot#13428

Merged
ebyhr merged 2 commits intotrinodb:masterfrom
elonazoulay:json_support
Aug 17, 2022
Merged

Add support for JSON type in Pinot#13428
ebyhr merged 2 commits intotrinodb:masterfrom
elonazoulay:json_support

Conversation

@elonazoulay
Copy link
Member

@elonazoulay elonazoulay commented Jul 31, 2022 •
edited by ebyhr
Loading

Description

Add json support to the Pinot connector.

Is this change a fix, improvement, new feature, refactoring, or other?

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

How would you describe this change to a non-technical end user or system administrator?

Related issues, pull requests, and links

Documentation

( ) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

( ) No release notes entries required.
(x) Release notes entries required with the following suggested text:

# Pinot
* Add support for `json` type. ({issue}`13428`)

@cla-bot cla-bot bot added the cla-signed label Jul 31, 2022
@elonazoulay elonazoulay requested review from hashhar and martint July 31, 2022 12:18
@ebyhr ebyhr changed the title Json support Add support for JSON type in Pinot Jul 31, 2022
ebyhr
ebyhr reviewed Aug 1, 2022
View reviewed changes
@ebyhr ebyhr mentioned this pull request Aug 1, 2022
Closed
@github-actions github-actions bot added the docs label Aug 2, 2022
@elonazoulay elonazoulay requested a review from ebyhr August 2, 2022 12:00
ebyhr
ebyhr reviewed Aug 3, 2022
View reviewed changes
@elonazoulay elonazoulay requested a review from ebyhr August 3, 2022 11:08
@ddcprg ddcprg mentioned this pull request Aug 3, 2022
Closed
ebyhr
ebyhr reviewed Aug 8, 2022
View reviewed changes
ebyhr
ebyhr approved these changes Aug 16, 2022
View reviewed changes
@ebyhr ebyhr merged commit 05a6295 into trinodb:master Aug 17, 2022
@ebyhr
Copy link
Member

ebyhr commented Aug 17, 2022

Merged, thanks!

@ebyhr ebyhr mentioned this pull request Aug 17, 2022
Closed
@github-actions github-actions bot added this to the 393 milestone Aug 17, 2022
@elonazoulay
Copy link
Member Author

elonazoulay commented Aug 17, 2022

thanks for the review, much appreciated @ebyhr !

@colebow colebow mentioned this pull request Aug 17, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ebyhr ebyhr ebyhr approved these changes

@hashhar hashhar Awaiting requested review from hashhar

@martint martint Awaiting requested review from martint

Assignees

No one assigned

Labels

cla-signed docs

Milestone

393

Development

Successfully merging this pull request may close these issues.

2 participants

@elonazoulay @ebyhr