Skip to content

Remove non-test uses of commons-lang3 #2100

@rdblue

Description

@rdblue

Iceberg should avoid unnecessary dependencies, like commons-*. A few dependencies on commons have snuck in:

[blue@work iceberg]$ ack 'org.apache.commons'
flink/src/main/java/org/apache/iceberg/flink/data/FlinkParquetReaders.java
29:import org.apache.commons.lang3.ArrayUtils;

versions.props
13:org.apache.commons:commons-lang3 = 3.9

pig/src/main/java/org/apache/iceberg/pig/IcebergPigInputFormat.java
30:import org.apache.commons.lang3.SerializationUtils;

spark3/src/main/java/org/apache/iceberg/spark/Spark3Util.java
28:import org.apache.commons.lang3.ArrayUtils;

We should remove those and move the dependency to testCompile.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions