Skip to content

Commit

Permalink
[hotfix] reallocate commons-codec in flink planner & blink planner
Browse files Browse the repository at this point in the history
  • Loading branch information
KurtYoung committed Jul 11, 2019
1 parent b0bf838 commit 64d5d69
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 0 deletions.
18 changes: 18 additions & 0 deletions flink-table/flink-table-planner-blink/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -328,6 +328,12 @@ under the License.
</includes>
</artifactSet>
<relocations>
<!-- Calcite is not relocated for now, because we expose it at some locations such as CalciteConfig -->
<!--<relocation>
<pattern>org.apache.calcite</pattern>
<shadedPattern>org.apache.flink.calcite.shaded.org.apache.calcite</shadedPattern>
</relocation>-->

<!-- Calcite's dependencies -->
<relocation>
<pattern>com.google</pattern>
Expand All @@ -341,6 +347,18 @@ under the License.
<pattern>com.fasterxml</pattern>
<shadedPattern>org.apache.flink.calcite.shaded.com.fasterxml</shadedPattern>
</relocation>
<relocation>
<pattern>org.apache.commons.codec</pattern>
<shadedPattern>org.apache.flink.calcite.shaded.org.apache.commons.codec</shadedPattern>
</relocation>

<!-- flink-table-planner dependencies -->
<!-- not relocated for now, because we need to change the contents of the properties field otherwise -->
<!--<relocation>
<pattern>org.codehaus</pattern>
<shadedPattern>org.apache.flink.table.shaded.org.codehaus</shadedPattern>
</relocation>-->

</relocations>
</configuration>
</execution>
Expand Down
4 changes: 4 additions & 0 deletions flink-table/flink-table-planner/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -364,6 +364,10 @@ under the License.
<pattern>com.fasterxml</pattern>
<shadedPattern>org.apache.flink.calcite.shaded.com.fasterxml</shadedPattern>
</relocation>
<relocation>
<pattern>org.apache.commons.codec</pattern>
<shadedPattern>org.apache.flink.calcite.shaded.org.apache.commons.codec</shadedPattern>
</relocation>

<!-- flink-table-planner dependencies -->
<relocation>
Expand Down

0 comments on commit 64d5d69

Please sign in to comment.