Skip to content

Commit f1f37e7

Browse files
committed
[MINOR] Update syntax and deprecated in docker
This commit remove the maxPemSize argument that is no longer valid in java, for the docker tests used in github actions.
1 parent c842072 commit f1f37e7

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

docker/entrypoint.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ cd /github/workspace/src/main/cpp
2626
./build.sh
2727
cd /github/workspace
2828

29-
export MAVEN_OPTS="-Xmx512m -XX:MaxPermSize=128m"
29+
export MAVEN_OPTS="-Xmx512m"
3030

3131
log="/tmp/sysdstest.log"
3232
mvn -ntp -B test-compile 2>&1 | grep -E "BUILD|Total time:|---|Building SystemDS"

docker/testsysds.Dockerfile

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,7 @@ RUN apt-get install -y --no-install-recommends \
7373
libssl-dev \
7474
r-base \
7575
r-base-dev \
76-
r-base-core\
77-
76+
r-base-core \
7877
&& Rscript installDependencies.R \
7978
&& rm -rf installDependencies.R \
8079
&& rm -rf /var/lib/apt/lists/*

0 commit comments

Comments
 (0)