This is an automated email from the ASF dual-hosted git repository.
jerrypeng pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 562cb36 reverting jackson version bump for sql (#2978)
562cb36 is described below
commit 562cb364ad5a56e2e3dd6c6950070357d513bf28
Author: Boyang Jerry Peng <[email protected]>
AuthorDate: Tue Nov 13 14:06:33 2018 -0800
reverting jackson version bump for sql (#2978)
* reverting jackson version bump for sql
* changing databind to be 2.8.11.1
* fix license file
---
pulsar-sql/pom.xml | 7 ++++--
pulsar-sql/presto-distribution/LICENSE | 26 +++++++++++-----------
pulsar-sql/presto-distribution/pom.xml | 7 ++++--
.../tests/integration/presto/TestBasicPresto.java | 8 +++++++
4 files changed, 31 insertions(+), 17 deletions(-)
diff --git a/pulsar-sql/pom.xml b/pulsar-sql/pom.xml
index 6b12118..bbb872d 100644
--- a/pulsar-sql/pom.xml
+++ b/pulsar-sql/pom.xml
@@ -32,7 +32,10 @@
<name>Pulsar SQL :: Parent</name>
<properties>
- <jackson.version>2.9.7</jackson.version>
+ <jackson.version>2.8.11</jackson.version>
+ <!--fix Security Vulnerabilities-->
+
<!--https://www.cvedetails.com/vulnerability-list/vendor_id-15866/product_id-42991/Fasterxml-Jackson-databind.html-->
+ <jackson.databind.version>2.8.11.1</jackson.databind.version>
</properties>
<modules>
@@ -58,7 +61,7 @@
<dependency>
<groupId>com.fasterxml.jackson.core</groupId>
<artifactId>jackson-databind</artifactId>
- <version>${jackson.version}</version>
+ <version>${jackson.databind.version}</version>
</dependency>
<dependency>
diff --git a/pulsar-sql/presto-distribution/LICENSE
b/pulsar-sql/presto-distribution/LICENSE
index 76d2034..866ba23 100644
--- a/pulsar-sql/presto-distribution/LICENSE
+++ b/pulsar-sql/presto-distribution/LICENSE
@@ -208,20 +208,19 @@ This projects includes binary packages with the following
licenses:
The Apache Software License, Version 2.0
* Jackson
- - jackson-annotations-2.9.7.jar
- - jackson-databind-2.9.7.jar
- - jackson-dataformat-smile-2.9.7.jar
- - jackson-datatype-guava-2.9.7.jar
- - jackson-datatype-guava-2.9.7.jar
- - jackson-datatype-jdk8-2.9.7.jar
- - jackson-datatype-jdk8-2.9.7.jar
- - jackson-datatype-joda-2.9.7.jar
- - jackson-datatype-jsr310-2.9.7.jar
- - jackson-core-2.9.7.jar
+ - jackson-annotations-2.8.11.jar
+ - jackson-databind-2.8.11.1.jar
+ - jackson-dataformat-smile-2.8.11.jar
+ - jackson-datatype-guava-2.8.11.jar
+ - jackson-datatype-guava-2.8.11.jar
+ - jackson-datatype-jdk8-2.8.11.jar
+ - jackson-datatype-jdk8-2.8.11.jar
+ - jackson-datatype-joda-2.8.11.jar
+ - jackson-datatype-jsr310-2.8.11.jar
+ - jackson-core-2.8.11.jar
- jackson-core-asl-1.9.13.jar
- - jackson-databind-2.9.7.jar
- jackson-mapper-asl-1.9.13.jar
- - jackson-dataformat-yaml-2.9.7.jar
+ - jackson-dataformat-yaml-2.8.11.jar
* Guava
- guava-21.0.jar
- guava-24.1-jre.jar
@@ -372,7 +371,8 @@ The Apache Software License, Version 2.0
* RocksDB JNI
- rocksdbjni-5.13.3.jar
* SnakeYAML
- - snakeyaml-1.23.jar
+ - snakeyaml-1.17.jar
+ - snakeyaml-1.23.jar
* Snappy Java
- snappy-java-1.1.1.3.jar
* Bean Validation API
diff --git a/pulsar-sql/presto-distribution/pom.xml
b/pulsar-sql/presto-distribution/pom.xml
index e05567a..e66d87c 100644
--- a/pulsar-sql/presto-distribution/pom.xml
+++ b/pulsar-sql/presto-distribution/pom.xml
@@ -42,7 +42,10 @@
<!-- Launcher properties -->
<main-class>com.facebook.presto.server.PrestoServer</main-class>
<process-name>${project.artifactId}</process-name>
- <jackson.version>2.9.7</jackson.version>
+ <jackson.version>2.8.11</jackson.version>
+ <!--fix Security Vulnerabilities-->
+
<!--https://www.cvedetails.com/vulnerability-list/vendor_id-15866/product_id-42991/Fasterxml-Jackson-databind.html-->
+ <jackson.databind.version>2.8.11.1</jackson.databind.version>
</properties>
<dependencies>
@@ -120,7 +123,7 @@
<dependency>
<groupId>com.fasterxml.jackson.core</groupId>
<artifactId>jackson-databind</artifactId>
- <version>${jackson.version}</version>
+ <version>${jackson.databind.version}</version>
</dependency>
<dependency>
diff --git
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/presto/TestBasicPresto.java
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/presto/TestBasicPresto.java
index c51b25f..505ada4 100644
---
a/tests/integration/src/test/java/org/apache/pulsar/tests/integration/presto/TestBasicPresto.java
+++
b/tests/integration/src/test/java/org/apache/pulsar/tests/integration/presto/TestBasicPresto.java
@@ -87,6 +87,14 @@ public class TestBasicPresto extends PulsarTestSuite {
producer.send(stock);
}
+ ContainerExecResult result = execQuery("show schemas in pulsar;");
+ assertThat(result.getExitCode()).isEqualTo(0);
+ assertThat(result.getStdout()).contains("public/default");
+
+ result = execQuery("show tables in pulsar.\"public/default\";");
+ assertThat(result.getExitCode()).isEqualTo(0);
+ assertThat(result.getStdout()).contains("stocks");
+
ContainerExecResult containerExecResult = execQuery("select * from
pulsar.\"public/default\".stocks order by entryid;");
assertThat(containerExecResult.getExitCode()).isEqualTo(0);
log.info("select sql query output \n{}",
containerExecResult.getStdout());
