Fix entry point and other things #2
Conversation
geoHeil
commented
Jan 28, 2017
geoHeil
commented
- fix entrypoint typo error
- fix geomesa jar in wrong folder / accumulo iterator class not found
- add a docker container for the tutorial. Not sure where that one should be included https://github.com/geoHeil/geomesa-tutorials/tree/master/docker-tutorial
|
Oh boy. This PR has a whole bunch of the stuff I needed and ran into painstakingly. Anything that needs to be done to merge it? |
|
|
||
| **step 0: clone the repos** | ||
|
|
||
| `git clone https://github.com/geoHeil/geodocker-geomesa.git` |
There was a problem hiding this comment.
If we are merging this, it'd be good for the docs to point to the GeoDocker repo.
| cd geomesa | ||
| git checkout geomesa_2.11-1.3.0 | ||
| update ./build.sh to desired version numbers then | ||
| build/mvn clean install -DskipTests=true #test failures otherwise (timeouts on macbook) |
There was a problem hiding this comment.
GeoMesa releases are available on LocationTech's Nexus repo. Would it be possible to skip this step?
|
|
||
| **step 2: build the image for the tutorial** | ||
| ``` | ||
| git clone https://github.com/geoHeil/geomesa-tutorials.git |
There was a problem hiding this comment.
Eh, we should point to the 'official' GeoMesa-Tutorials repo.
| - currently it is crashing | ||
| - checkout geomesas gitter, there are some really interesting notes | ||
| - apparently the iterators ore (not yet out of the box) available in the geodocker implementation | ||
| - you need to add the jar into `ACCUMULO_HOME/lib` to *globally* install it. Otherwise namespaces are possible as well |
There was a problem hiding this comment.
Testing locally is definitely good. Did you have a chance to figure out some of these options?
| @@ -2,8 +2,8 @@ | |||
|
|
|||
| usage() { echo "Usage: $0 [-t <tag>] [-g <geomesa.version>] [-a <accumulo.version>]" 1>&2; exit 1; } | |||
|
|
|||
| ACCUMULO_VERSION=1.7.2 | |||
| GEOMESA_VERSION=1.3.0-m3-SNAPSHOT | |||
| ACCUMULO_VERSION=1.8.0 | |||
There was a problem hiding this comment.
My preference would be to stick to Accumulo 1.7.2. GeoMesa 1.3.x is built against it. User needing to use Accumulo 1.8.0 need to build GeoMesa with a profile.
