Conversation
.travis.yml
Outdated
| addons: | ||
| apt: | ||
| packages: | ||
| - python3 |
There was a problem hiding this comment.
It seems that Python3 should be preinstalled on trusty image: https://docs.travis-ci.com/user/trusty-ci-environment/#Python. Do we need it here?
There was a problem hiding this comment.
You are right, I have not tried to remove these flags since moving to trusty.
| matrix: | ||
| exclude: | ||
| - scala: 2.10.5 # Spark 2.0.0 pre-built with Scala 2.11 in the tgz download | ||
| env: SPARK_VERSION=2.0.1 SPARK_BUILD="spark-2.0.1-bin-hadoop2.7" SPARK_BUILD_URL="http://d3kbcqa49mib13.cloudfront.net/spark-2.0.1-bin-hadoop2.7.tgz" |
There was a problem hiding this comment.
Is it simpler if we don't list scala as the main language and let the test line be sbt -Dscala.version=2.10.6, e.g.?
There was a problem hiding this comment.
Btw, shall we also bump scala version from 2.10.5 to 2.10.6?
There was a problem hiding this comment.
I am going to try it out.
|
|
||
| install: | ||
| - pip install --user -r ./python/requirements.txt | ||
| - pip3 install --user -r ./python/requirements.txt |
There was a problem hiding this comment.
We might use pyenv to switch python version first and make this line the same for py2 and py3.
There was a problem hiding this comment.
I suggest we keep it this way for now. It just takes a few seconds and it is simpler to install both.
| - oraclejdk7 # openJDK crashes sometimes | ||
| sudo: false | ||
|
|
||
| sudo: required |
There was a problem hiding this comment.
If we no longer need to install pip, does it mean we don't need sudo either?
There was a problem hiding this comment.
We do. trusty is beta, and fails (jvm package not properly installed)
|
That's a great suggestion. Yes, we should tag the ticket in the title. On Fri, Nov 11, 2016 at 8:50 AM, Nicholas Chammas [email protected]
|
3 participants
No description provided.