Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Situ ES instance optional #1

Open
mikeiannacone opened this issue Apr 26, 2018 · 0 comments
Open

Make Situ ES instance optional #1

mikeiannacone opened this issue Apr 26, 2018 · 0 comments

Comments

@mikeiannacone
Copy link
Member

Right now, the SITU_DB_CONFIG env var (and a valid Elasticsearch instance) is needed, even if no situ instance is present.

It should be a relatively easy fix to make this optional & configurable, instead of assuming it is present. The check here will need to be changed, and the querySitu , getVertexSitu , getInEdgesFromSitu, and getOutEdgesFromSitu methods in the DBConnectionResource.java will need to be modified to return empty results when not connected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant