Skip to content

Fix entry point and other things#2

Open
geoHeil wants to merge 11 commits intogeodocker:masterfrom
geoHeil:fixEntryPoint
Open

Fix entry point and other things#2
geoHeil wants to merge 11 commits intogeodocker:masterfrom
geoHeil:fixEntryPoint

Conversation

@geoHeil
Copy link

@geoHeil geoHeil commented Jan 28, 2017

@rburhum
Copy link

rburhum commented May 12, 2017

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`
Copy link
Collaborator

Choose a reason for hiding this comment

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

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)
Copy link
Collaborator

Choose a reason for hiding this comment

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

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
Copy link
Collaborator

Choose a reason for hiding this comment

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

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
Copy link
Collaborator

Choose a reason for hiding this comment

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

Testing locally is definitely good. Did you have a chance to figure out some of these options?


ACCUMULO_VERSION=1.7.2
GEOMESA_VERSION=1.3.0-m3-SNAPSHOT
ACCUMULO_VERSION=1.8.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

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.

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.

3 participants