Skip to content

Commit

Permalink
Update Swagger to 1.6.2 and Maven Swagger Plugin to 3.1.7 (apache#8845)
Browse files Browse the repository at this point in the history
### Motivation

In out build environment (MagNews.com) we are upgrading Swagger to 1.6.2, as it is also used by Pulsar Builder system we would like to upgrade Swagger here as well.
Even if Swagger is used for documentation, swagger-core is a runtime dependency so it is better to guarantee that Pulsar is working well with latest version of Swagger 1.X.

I am also upgrading the Maven Swagger Plugin to 3.1.7, the 3.1.8 is buggy and it fails the build with NPE (the same problem occurs even in MagNews build, so it is not related to Pulsar itself)

### Modifications
- Upgrade Swagger version to 1.6.2
- Upgrade Maven Plugin to 3.1.7
  • Loading branch information
eolivelli authored Dec 9, 2020
1 parent ac845aa commit 9953129
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions distribution/server/src/assemble/LICENSE.bin.txt
Original file line number Diff line number Diff line change
Expand Up @@ -339,9 +339,9 @@ The Apache Software License, Version 2.0
* J2ObjC Annotations -- com.google.j2objc-j2objc-annotations-1.3.jar
* Netty Reactive Streams -- com.typesafe.netty-netty-reactive-streams-2.0.4.jar
* Swagger
- io.swagger-swagger-annotations-1.5.21.jar
- io.swagger-swagger-core-1.5.21.jar
- io.swagger-swagger-models-1.5.21.jar
- io.swagger-swagger-annotations-1.6.2.jar
- io.swagger-swagger-core-1.6.2.jar
- io.swagger-swagger-models-1.6.2.jar
* DataSketches
- com.yahoo.datasketches-memory-0.8.3.jar
- com.yahoo.datasketches-sketches-core-0.8.3.jar
Expand Down
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ flexible messaging model and an intuitive client API.</description>
<jackson.version>2.11.1</jackson.version>
<jackson.databind.version>2.11.1</jackson.databind.version>
<reflections.version>0.9.11</reflections.version>
<swagger.version>1.5.21</swagger.version>
<swagger.version>1.6.2</swagger.version>
<puppycrawl.checkstyle.version>8.37</puppycrawl.checkstyle.version>
<dockerfile-maven.version>1.4.13</dockerfile-maven.version>
<typetools.version>0.5.0</typetools.version>
Expand Down
2 changes: 1 addition & 1 deletion pulsar-broker/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -445,7 +445,7 @@
<plugin>
<groupId>com.github.kongchen</groupId>
<artifactId>swagger-maven-plugin</artifactId>
<version>3.1.5</version>
<version>3.1.7</version>
<configuration>
<apiSources>
<apiSource>
Expand Down
2 changes: 1 addition & 1 deletion pulsar-sql/presto-distribution/LICENSE
Original file line number Diff line number Diff line change
Expand Up @@ -459,7 +459,7 @@ The Apache Software License, Version 2.0
* Apache Yetus Audience Annotations
- audience-annotations-0.5.0.jar
* Swagger
- swagger-annotations-1.5.21.jar
- swagger-annotations-1.6.2.jar

Protocol Buffers License
* Protocol Buffers
Expand Down

0 comments on commit 9953129

Please sign in to comment.