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.

Sign up for GitHub

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

Jump to bottom

[YCQL] toJSON() built-in must support UDT & FROZEN #1675

Closed
OlegLoginov opened this issue Jul 2, 2019 · 0 comments
Closed

[YCQL] toJSON() built-in must support UDT & FROZEN #1675

OlegLoginov opened this issue Jul 2, 2019 · 0 comments
Assignees
@OlegLoginov
Labels
area/ycql Yugabyte CQL (YCQL) kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue

Comments

@OlegLoginov
Copy link
Contributor

OlegLoginov commented Jul 2, 2019 •
edited by yugabyte-ci
Loading

Jira Link: DB-4704
This is a part of #992: [YCQL] add support for toJSON(), fromJSON() built-ins #992
Current implementation support all simple types and collections.
User-defined-type and frozen types are not supported yet.
@rkarthik007 rkarthik007 added the kind/enhancement This is an enhancement of an existing feature label Jul 2, 2019
OlegLoginov added a commit that referenced this issue Aug 5, 2019
@OlegLoginov
#1675: Support UDT & FROZEN in ToJson() built-in function.
da6aaf2 
Summary:
Fixed: [YCQL] toJSON() built-in must support UDT & FROZEN
#1675

Test Plan:
ybd --java-test org.yb.cql.TestSelect#testToJson
ybd --cxx-test ql-select-expr-test --gtest_filter QLTestSelectedExpr.TestQLSelectToJson
ybd --cxx-test ql-insert-table-test --gtest_filter TestQLInsertTable.TestQLInsertToJson
ybd --cxx-test bfql-test --gtest_filter BfqlTest.TestBuiltinToJson

```
Types/Values           |           Yugabyte |          Cassandra |
------------------------------------------------------------------
UDT                    |     {"v1":1,"v2":2}|  {"v1": 1, "v2": 2}|
FROZEN<type>           |      same as <type>|      same as <type>|
```
For example: MAP<int, FROZEN<UDT>>
Cassandra:    {"44": {"v1": 66, "v2": 77}, "99": {"v1": 11, "v2": 22}}
YB:                 {"44":{"v1":66,"v2":77},"99":{"v1":11,"v2":22}}

Reviewers: alex, neil

Reviewed By: neil

Subscribers: yql, kannan, eng

Differential Revision: https://phabricator.dev.yugabyte.com/D6855
@rthallamko3 rthallamko3 added the area/ycql Yugabyte CQL (YCQL) label Dec 29, 2022
@yugabyte-ci yugabyte-ci added the priority/medium Medium priority issue label Dec 29, 2022
@m-iancu m-iancu closed this as completed Jan 17, 2023
@github-project-automation github-project-automation bot moved this to Done in YQL-beta Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@OlegLoginov OlegLoginov

Labels
area/ycql Yugabyte CQL (YCQL) kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue
Projects
YQL-beta
Status: Done
Milestone
No milestone
Development

No branches or pull requests
5 participants
@m-iancu @rkarthik007 @rthallamko3 @yugabyte-ci @OlegLoginov