flink-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aljos...@apache.org
Subject flink git commit: [FLINK-7133] Exclude optional asm deps from Elasticsearch base module
Date Tue, 11 Jul 2017 15:06:28 GMT
Repository: flink
Updated Branches:
  refs/heads/release-1.3 8bfd1206f -> ad8766e10


[FLINK-7133] Exclude optional asm deps from Elasticsearch base module

These ASM dependencies where shaded into the elasticsearch-base module
which where then clashing with our newer (also shaded) ASM dependency.


Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/ad8766e1
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/ad8766e1
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/ad8766e1

Branch: refs/heads/release-1.3
Commit: ad8766e10031d813cc0dde82d73d0c8d02f47261
Parents: 8bfd120
Author: adebski <andrzej.debski91@gmail.com>
Authored: Sat Jul 8 17:03:18 2017 +0200
Committer: Aljoscha Krettek <aljoscha.krettek@gmail.com>
Committed: Tue Jul 11 17:05:58 2017 +0200

----------------------------------------------------------------------
 .../flink-connector-elasticsearch-base/pom.xml         | 13 +++++++++++++
 1 file changed, 13 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flink/blob/ad8766e1/flink-connectors/flink-connector-elasticsearch-base/pom.xml
----------------------------------------------------------------------
diff --git a/flink-connectors/flink-connector-elasticsearch-base/pom.xml b/flink-connectors/flink-connector-elasticsearch-base/pom.xml
index 32327ff..cb166cf 100644
--- a/flink-connectors/flink-connector-elasticsearch-base/pom.xml
+++ b/flink-connectors/flink-connector-elasticsearch-base/pom.xml
@@ -55,6 +55,19 @@ under the License.
 			<groupId>org.elasticsearch</groupId>
 			<artifactId>elasticsearch</artifactId>
 			<version>${elasticsearch.version}</version>
+			<!--
+			FLINK-7133: Excluding all org.ow2.asm from elasticsearch dependencies because
+			1. from the POV of client they are optional,
+			2. the version configured by default at the time of writing this comment (1.7.1) depends
on asm 4.1
+			   and when it is shaded into elasticsearch-base artifact it conflicts with newer shaded
versions of asm
+			   resulting in errors at the runtime when application is executed locally, e.g. from
IDE.
+			-->
+			<exclusions>
+				<exclusion>
+					<groupId>org.ow2.asm</groupId>
+					<artifactId>*</artifactId>
+				</exclusion>
+			</exclusions>
 		</dependency>
 
 		<!-- test dependencies -->


Mime
View raw message