Skip to content

CircleCI - #3236

Merged
pomadchin merged 60 commits into
locationtech:masterfrom
pomadchin:feature/circle-ci
Apr 28, 2020
Merged

pomadchin merged 60 commits into
locationtech:masterfrom
pomadchin:feature/circle-ci

Conversation

@pomadchin

Copy link
Copy Markdown
Member

Overview

This PR replaces Travis with CircleCI. It allows to speedup the tests run and purifies some parts of the CI environment (for instance HBase testing).

@pomadchin
pomadchin requested a review from echeipesh April 27, 2020 17:22
@pomadchin pomadchin self-assigned this Apr 27, 2020
Comment thread .circleci/config.yml
name: Artifacts publish
command: |
export SCALA_VERSION=<< parameters.scala-version >>
./sbt -Dsbt.supershell=false "++$SCALA_VERSION" publishLocal

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should eventually become publish and not publishLocal

ArrayTile(array, rasterExtent.cols, rasterExtent.rows)
}
def rasterizeWithValue(geom: Geometry, rasterExtent: RasterExtent, value: Int): Tile =
if(!geom.isValid) throw new IllegalArgumentException("Cannot rasterize an invalid polygon")

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the geometry is topologically invalid, it makes no sense to rasterize it?

def stddev(xs: List[Int], avg: Double): Double = xs match {
case Nil => Double.NaN
case ys => math.sqrt((0.0 /: ys) {
case ys => math.sqrt(ys.foldLeft(0.0) {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/: is deprecated.

./sbt -Dsbt.supershell=false "++$SCALA_VERSION" \
"project spark" test \
"project spark-pipeline" test && \
./sbt -Dsbt.supershell=false "++$SCALA_VERSION" \

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Splited into separate sbt starts due to memory issues.

@pomadchin
pomadchin force-pushed the feature/circle-ci branch 4 times, most recently from 249032d to bc76d4a Compare April 27, 2020 21:34
@pomadchin pomadchin changed the title [WIP] CircleCI CircleCI Apr 27, 2020
@pomadchin
pomadchin merged commit 83bd8e5 into locationtech:master Apr 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant