Skip to content

Commit

Permalink
[#427] Change CQL wire protocol to move the JSONB type
Browse files Browse the repository at this point in the history
Summary:
Since JSONB is not yet supported by Apache Cassandra, we would like to
move JSONB to a enum value which is far away from the existing range so
as to not conflict with future types added by Cassandra.

Test Plan: Jenkins, org.yb.cql.TestJson

Reviewers: mihnea, robert

Reviewed By: robert

Subscribers: kannan, yql

Differential Revision: https://phabricator.dev.yugabyte.com/D5264
  • Loading branch information
rven1 committed Aug 9, 2018
1 parent b5d66ea commit 9d0085a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion README.md
Expand Up @@ -165,7 +165,7 @@ using our driver by adding the following Maven dependency to your application:
<dependency>
<groupId>com.yugabyte</groupId>
<artifactId>cassandra-driver-core</artifactId>
<version>3.2.0-yb-16</version>
<version>3.2.0-yb-17</version>
</dependency>
```

Expand Down
4 changes: 2 additions & 2 deletions java/pom.xml
Expand Up @@ -76,8 +76,8 @@

<!-- Library dependencies -->
<async.version>1.4.1</async.version>
<cassandra-driver-core.version>3.2.0-yb-16</cassandra-driver-core.version>
<spark-cassandra-connector.version>2.0.5-yb-6</spark-cassandra-connector.version>
<cassandra-driver-core.version>3.2.0-yb-17</cassandra-driver-core.version>
<spark-cassandra-connector.version>2.0.5-yb-7</spark-cassandra-connector.version>
<lz4.version>1.3.0</lz4.version>
<snappy.version>1.1.4</snappy.version>
<commons-cli.version>1.2</commons-cli.version>
Expand Down
4 changes: 2 additions & 2 deletions src/yb/yql/cql/cqlserver/cql_message.h
Expand Up @@ -622,12 +622,12 @@ class ResultResponse : public CQLResponse {
TIME = 0x0012, // Since V4
SMALLINT = 0x0013, // Since V4
TINYINT = 0x0014, // Since V4
JSONB = 0x0019,
LIST = 0x0020,
MAP = 0x0021,
SET = 0x0022,
UDT = 0x0030,
TUPLE = 0x0031
TUPLE = 0x0031,
JSONB = 0x0080 // Yugabyte specific types start here
};

Id id = Id::CUSTOM;
Expand Down

0 comments on commit 9d0085a

Please sign in to comment.