Skip to content
This repository has been archived by the owner on Oct 26, 2022. It is now read-only.

Commit

Permalink
Fix TODO item
Browse files Browse the repository at this point in the history
  • Loading branch information
nielsbasjes committed Dec 14, 2020
1 parent 167423a commit b35caa4
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 11 deletions.
10 changes: 6 additions & 4 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -453,8 +453,9 @@
<target>${target.java.version}</target>
<compilerArgs>
<!-- Enable almost all warnings.-->
<!-- FIXME: Why does the hive udf fail with "An unknown compilation problem occurred" when using -Xlint:classfile,processing ? -->
<!-- Ignore the 'serial' warnings about not having a serialVersionUID -->
<!-- Ignore 'classfile' to allow leaving Kryo out. -->
<!-- Ignore 'processing' to avoid "No processor claimed any of these annotations" in several places. -->
<!-- Ignore 'serial' warnings about not having a serialVersionUID -->
<arg>-Xlint:all,-classfile,-processing,-serial</arg>
<arg>-Werror</arg>
</compilerArgs>
Expand All @@ -480,8 +481,9 @@
<release>${target.java.release}</release>
<compilerArgs>
<!-- Enable almost all warnings.-->
<!-- FIXME: Why does the hive udf fail with "An unknown compilation problem occurred" when using -Xlint:classfile,processing ? -->
<!-- Ignore the 'serial' warnings about not having a serialVersionUID -->
<!-- Ignore 'classfile' to allow leaving Kryo out. -->
<!-- Ignore 'processing' to avoid "No processor claimed any of these annotations" in several places. -->
<!-- Ignore 'serial' warnings about not having a serialVersionUID -->
<arg>-Xlint:all,-classfile,-processing,-serial</arg>
<arg>-Werror</arg>
<arg>-implicit:class</arg>
Expand Down
7 changes: 0 additions & 7 deletions udfs/beam/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -69,13 +69,6 @@
<scope>test</scope>
</dependency>

<dependency>
<groupId>com.esotericsoftware</groupId>
<artifactId>kryo</artifactId>
<version>${kryo5.version}</version>
<scope>provided</scope>
</dependency>

<!--=========================-->
<!--Needed for PAssert-->
<dependency>
Expand Down

0 comments on commit b35caa4

Please sign in to comment.