This is an automated email from the ASF dual-hosted git repository.
cwylie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git
The following commit(s) were added to refs/heads/master by this push:
new d00a967 Set aws.region for unit tests automatically (#7868)
d00a967 is described below
commit d00a9676b7da0fa9400e657bfae8a2961ca9de15
Author: Jihoon Son <jihoon...@apache.org>
AuthorDate: Fri Jun 14 15:34:22 2019 -0700
Set aws.region for unit tests automatically (#7868)
* Set aws.region for unit tests automatically
* Update README.template
---
.travis.yml | 2 --
README.template | 2 --
pom.xml | 2 ++
3 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index be35906..8ccad05 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -97,7 +97,6 @@ matrix:
# other modules test
- env:
- NAME="other modules test"
- - AWS_REGION=us-east-1 # set a aws region for unit tests
install: MAVEN_OPTS='-Xmx3000m' mvn install -q -ff -DskipTests -B
before_script: unset _JAVA_OPTIONS
script:
@@ -109,7 +108,6 @@ matrix:
# other modules test with SQL Compatibility enabled
- env:
- NAME="other modules test with SQL Compatibility"
- - AWS_REGION=us-east-1 # set a aws region for unit tests
install: MAVEN_OPTS='-Xmx3000m' mvn install -q -ff -DskipTests -B
before_script: unset _JAVA_OPTIONS
script:
diff --git a/README.template b/README.template
index fbf4d8d..36839c7 100644
--- a/README.template
+++ b/README.template
@@ -62,8 +62,6 @@ licenses, and skip the unit tests, you would run:
mvn clean install -Papache-release,dist,rat -DskipTests
-Note: the AWS S3 unit tests require the 'AWS_DEFAULT_REGION' environment
variable to be set to function correctly.
-
Community
---------
diff --git a/pom.xml b/pom.xml
index e7ad47a..21a5d2c 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1228,6 +1228,7 @@
-Dfile.encoding=UTF-8
-Duser.timezone=UTC
-Djava.util.logging.manager=org.apache.logging.log4j.jul.LogManager
+ -Daws.region=us-east-1 <!-- required for
s3-related unit tests -->
<!--@TODO After fixing
https://github.com/apache/incubator-druid/issues/4964 remove this parameter-->
-Ddruid.indexing.doubleStorage=double
</argLine>
@@ -1490,6 +1491,7 @@
<!-- set heap size to work around
https://github.com/travis-ci/travis-ci/issues/3396 -->
<argLine>-Xmx768m -Duser.language=en
-Duser.country=US -Dfile.encoding=UTF-8
-Duser.timezone=UTC
-Djava.util.logging.manager=org.apache.logging.log4j.jul.LogManager
+ -Daws.region=us-east-1 <!-- required for
s3-related unit tests -->
<!--@TODO After fixing
https://github.com/apache/incubator-druid/issues/4964 remove this parameter-->
-Ddruid.indexing.doubleStorage=double
</argLine>
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org
