Re-enable datavec-geo and datavec-hadoop as module in pom.xml; update imports (#425)

* Re-enable datavec-geo and datavec-hadoop as module in pom.xml; update imports

Signed-off-by: Alex Black <blacka101@gmail.com>

* Fix parent pom and cyclical dependency

Signed-off-by: Alex Black <blacka101@gmail.com>

datavec/datavec-data/datavec-data-nlp/pom.xml

@@ -67,6 +67,13 @@
             <version>${nd4j.version}</version>
             <scope>test</scope>
         </dependency>
+
+        <dependency>
+            <groupId>org.datavec</groupId>
+            <artifactId>datavec-local</artifactId>
+            <version>${project.version}</version>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 
     <profiles>

datavec/datavec-data/datavec-geo/pom.xml

@@ -18,7 +18,7 @@
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
     <parent>
-        <artifactId>datavec-parent</artifactId>
+        <artifactId>datavec-data</artifactId>
         <groupId>org.datavec</groupId>
         <version>1.0.0-SNAPSHOT</version>
     </parent>

datavec/datavec-data/datavec-geo/src/main/java/org/datavec/api/transform/reduce/geo/CoordinatesReduction.java

@@ -27,7 +27,7 @@ import org.datavec.api.transform.schema.Schema;
 import org.datavec.api.writable.DoubleWritable;
 import org.datavec.api.writable.Text;
 import org.datavec.api.writable.Writable;
-import org.nd4j.linalg.function.Supplier;
+import org.nd4j.common.function.Supplier;
 
 import java.util.ArrayList;
 import java.util.Collections;

datavec/datavec-data/datavec-geo/src/main/java/org/datavec/api/transform/transform/geo/GeoIPFetcher.java

@@ -17,8 +17,8 @@
 package org.datavec.api.transform.transform.geo;
 
 import org.apache.commons.io.FileUtils;
-import org.nd4j.base.Preconditions;
+import org.nd4j.common.base.Preconditions;
-import org.nd4j.util.ArchiveUtils;
+import org.nd4j.common.util.ArchiveUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 

datavec/datavec-data/datavec-geo/src/test/java/org/datavec/api/transform/AssertTestsExtendBaseClass.java

@@ -16,9 +16,10 @@
 package org.datavec.api.transform;
 
 import lombok.extern.slf4j.Slf4j;
+import org.nd4j.common.tests.AbstractAssertTestsClass;
+import org.nd4j.common.tests.BaseND4JTest;
+
 import java.util.*;
-import org.nd4j.AbstractAssertTestsClass;
-import org.nd4j.BaseND4JTest;
 
 /**
  * This class checks that all test classes (i.e., anything with one or more methods annotated with @Test)

datavec/datavec-data/datavec-hadoop/pom.xml

@@ -18,7 +18,7 @@
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
     <parent>
-        <artifactId>datavec-parent</artifactId>
+        <artifactId>datavec-data</artifactId>
         <groupId>org.datavec</groupId>
         <version>1.0.0-SNAPSHOT</version>
     </parent>

datavec/datavec-data/datavec-hadoop/src/main/java/org/datavec/hadoop/records/reader/mapfile/IndexToKey.java

@@ -19,7 +19,7 @@ package org.datavec.hadoop.records.reader.mapfile;
 import org.apache.hadoop.io.MapFile;
 import org.apache.hadoop.io.Writable;
 import org.apache.hadoop.io.WritableComparable;
-import org.nd4j.linalg.primitives.Pair;
+import org.nd4j.common.primitives.Pair;
 
 import java.io.IOException;
 import java.util.List;

datavec/datavec-data/datavec-hadoop/src/main/java/org/datavec/hadoop/records/reader/mapfile/MapFileReader.java

@@ -23,9 +23,9 @@ import org.apache.hadoop.io.SequenceFile;
 import org.apache.hadoop.io.Writable;
 import org.apache.hadoop.io.WritableComparable;
 import org.apache.hadoop.util.ReflectionUtils;
-import org.nd4j.linalg.primitives.Pair;
 import org.datavec.hadoop.records.reader.mapfile.index.LongIndexToKey;
 import org.datavec.hadoop.records.reader.mapfile.record.RecordWritable;
+import org.nd4j.common.primitives.Pair;
 
 import java.io.Closeable;
 import java.io.IOException;

datavec/datavec-data/datavec-hadoop/src/main/java/org/datavec/hadoop/records/reader/mapfile/MapFileRecordReader.java

@@ -26,7 +26,7 @@ import org.datavec.api.split.InputSplit;
 import org.datavec.api.writable.Writable;
 import org.datavec.hadoop.records.reader.mapfile.index.LongIndexToKey;
 import org.datavec.hadoop.records.reader.mapfile.record.RecordWritable;
-import org.nd4j.linalg.util.MathUtils;
+import org.nd4j.common.util.MathUtils;
 
 import java.io.DataInputStream;
 import java.io.File;

datavec/datavec-data/datavec-hadoop/src/main/java/org/datavec/hadoop/records/reader/mapfile/MapFileSequenceRecordReader.java

@@ -28,7 +28,7 @@ import org.datavec.api.split.InputSplit;
 import org.datavec.api.writable.Writable;
 import org.datavec.hadoop.records.reader.mapfile.index.LongIndexToKey;
 import org.datavec.hadoop.records.reader.mapfile.record.SequenceRecordWritable;
-import org.nd4j.linalg.util.MathUtils;
+import org.nd4j.common.util.MathUtils;
 
 import java.io.DataInputStream;
 import java.io.File;

datavec/datavec-data/datavec-hadoop/src/main/java/org/datavec/hadoop/records/reader/mapfile/index/LongIndexToKey.java

@@ -18,11 +18,10 @@ package org.datavec.hadoop.records.reader.mapfile.index;
 
 import org.apache.hadoop.io.LongWritable;
 import org.apache.hadoop.io.MapFile;
-import org.apache.hadoop.io.SequenceFile;
 import org.apache.hadoop.io.Writable;
 import org.apache.hadoop.util.ReflectionUtils;
-import org.nd4j.linalg.primitives.Pair;
 import org.datavec.hadoop.records.reader.mapfile.IndexToKey;
+import org.nd4j.common.primitives.Pair;
 
 import java.io.IOException;
 import java.util.ArrayList;

datavec/datavec-data/datavec-hadoop/src/test/java/org/datavec/hadoop/AssertTestsExtendBaseClass.java

@@ -16,9 +16,10 @@
 package org.datavec.hadoop;
 
 import lombok.extern.slf4j.Slf4j;
+import org.nd4j.common.tests.AbstractAssertTestsClass;
+import org.nd4j.common.tests.BaseND4JTest;
+
 import java.util.*;
-import org.nd4j.AbstractAssertTestsClass;
-import org.nd4j.BaseND4JTest;
 /**
  * This class checks that all test classes (i.e., anything with one or more methods annotated with @Test)
  * extends BaseND4jTest - either directly or indirectly.

datavec/datavec-data/datavec-hadoop/src/test/java/org/datavec/hadoop/records/reader/TestMapFileRecordReader.java

@@ -16,6 +16,7 @@
 
 package org.datavec.hadoop.records.reader;
 
+import org.nd4j.common.util.MathUtils;
 import org.nd4j.shade.guava.io.Files;
 import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.fs.Path;
@@ -36,7 +37,6 @@ import org.junit.AfterClass;
 import org.junit.BeforeClass;
 import org.junit.Test;
 import org.nd4j.linalg.factory.Nd4j;
-import org.nd4j.linalg.util.MathUtils;
 
 import java.io.File;
 import java.io.IOException;

datavec/datavec-data/datavec-hadoop/src/test/java/org/datavec/hadoop/records/reader/TestMapFileRecordReaderMultipleParts.java

@@ -16,6 +16,8 @@
 
 package org.datavec.hadoop.records.reader;
 
+import org.nd4j.common.primitives.Pair;
+import org.nd4j.common.util.MathUtils;
 import org.nd4j.shade.guava.io.Files;
 import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.fs.Path;
@@ -36,8 +38,6 @@ import org.datavec.hadoop.records.reader.mapfile.record.SequenceRecordWritable;
 import org.junit.AfterClass;
 import org.junit.BeforeClass;
 import org.junit.Test;
-import org.nd4j.linalg.primitives.Pair;
-import org.nd4j.linalg.util.MathUtils;
 
 import java.io.File;
 import java.io.IOException;

datavec/datavec-data/datavec-hadoop/src/test/java/org/datavec/hadoop/records/reader/TestMapFileRecordReaderMultiplePartsSomeEmpty.java

@@ -16,6 +16,8 @@
 
 package org.datavec.hadoop.records.reader;
 
+import org.nd4j.common.primitives.Pair;
+import org.nd4j.common.util.MathUtils;
 import org.nd4j.shade.guava.io.Files;
 import org.apache.hadoop.conf.Configuration;
 import org.apache.hadoop.fs.Path;
@@ -36,8 +38,6 @@ import org.datavec.hadoop.records.reader.mapfile.record.SequenceRecordWritable;
 import org.junit.AfterClass;
 import org.junit.BeforeClass;
 import org.junit.Test;
-import org.nd4j.linalg.primitives.Pair;
-import org.nd4j.linalg.util.MathUtils;
 
 import java.io.File;
 import java.io.IOException;

datavec/datavec-data/pom.xml

@@ -33,6 +33,8 @@
         <module>datavec-data-codec</module>
         <module>datavec-data-image</module>
         <module>datavec-data-nlp</module>
+        <module>datavec-geo</module>
+        <module>datavec-hadoop</module>
     </modules>
 
     <properties>
@@ -45,12 +47,6 @@
             <artifactId>nd4j-api</artifactId>
             <version>${nd4j.version}</version>
         </dependency>
-        <dependency>
-            <groupId>org.datavec</groupId>
-            <artifactId>datavec-local</artifactId>
-            <version>${project.version}</version>
-            <scope>test</scope>
-        </dependency>
     </dependencies>
 
     <profiles>