You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ks...@apache.org on 2018/11/22 18:15:02 UTC

[arrow] branch master updated: ARROW-3756: [CI/Docker/Java] Java tests are failing in docker-compose setup

This is an automated email from the ASF dual-hosted git repository.

kszucs pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new dc6da3a  ARROW-3756: [CI/Docker/Java] Java tests are failing in docker-compose setup
dc6da3a is described below

commit dc6da3abea1ad72d0a26d8d3f6e69b2b359f3cc4
Author: Krisztián Szűcs <sz...@gmail.com>
AuthorDate: Thu Nov 22 19:14:42 2018 +0100

    ARROW-3756: [CI/Docker/Java] Java tests are failing in docker-compose setup
    
    Don't merge yet, the tests are still failing.
    
    Author: Krisztián Szűcs <sz...@gmail.com>
    
    Closes #2941 from kszucs/ARROW-3756 and squashes the following commits:
    
    461f8035 <Krisztián Szűcs> mount local m2 repo; use maven 3.5.2 image
    4fc33d02 <Krisztián Szűcs> alternative build directory
    a004f826 <Krisztián Szűcs> use the mounted docker_cache folder as maven repository; set forkCount to zero
---
 docker-compose.yml |  6 ++++++
 java/Dockerfile    | 10 +---------
 java/pom.xml       |  6 +++---
 3 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/docker-compose.yml b/docker-compose.yml
index a62eb33..50e4dde 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -63,10 +63,16 @@ services:
     volumes: *volumes
 
   java:
+    # Usage:
+    #   docker-compose build java
+    #   docker-compose run java
     image: arrow:java
     build:
       context: .
       dockerfile: java/Dockerfile
+    volumes:
+      - .:/arrow:delegated
+      - $HOME/.m2:/root/.m2:delegated
 
   js:
     image: arrow:js
diff --git a/java/Dockerfile b/java/Dockerfile
index 96c4d4c..4ef9d28 100644
--- a/java/Dockerfile
+++ b/java/Dockerfile
@@ -15,16 +15,8 @@
 # specific language governing permissions and limitations
 # under the License.
 
-FROM maven
+FROM maven:3.5.2-jdk-8-slim
 
-ADD header /arrow/
-ADD format /arrow/format
-ADD integration/data /arrow/integration/data
-ADD java /arrow/java
 WORKDIR /arrow/java
 
-# build
-RUN mvn -DskipTests=true -Dcheckstyle.skip=true -B install
-
-# test
 CMD mvn test
diff --git a/java/pom.xml b/java/pom.xml
index f6847ae..0df1178 100644
--- a/java/pom.xml
+++ b/java/pom.xml
@@ -636,11 +636,11 @@
     <profile>
       <id>java-8</id>
       <activation>
-	<jdk>[1.8,)</jdk>
+        <jdk>[1.8,)</jdk>
       </activation>
       <properties>
-	<doclint>none</doclint>
-	<additionalparam>-Xdoclint:none</additionalparam>
+        <doclint>none</doclint>
+        <additionalparam>-Xdoclint:none</additionalparam>
       </properties>
     </profile>