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

index aanmaken met mapping werkt niet op amazon #72

Closed
wires opened this issue Oct 21, 2015 · 11 comments
Closed

index aanmaken met mapping werkt niet op amazon #72

wires opened this issue Oct 21, 2015 · 11 comments

Comments

@wires
Copy link
Contributor

wires commented Oct 21, 2015

@bertspaan NOOIT ERRORS OPETEN

fix:
https://gist.github.com/wires/6aa3615cc99f47693fd5#file-default-mapping-js-L35

lowercase analyser was unknown..

wires added a commit to histograph/io that referenced this issue Oct 21, 2015
@bertspaan
Copy link
Member

😢

@wires
Copy link
Contributor Author

wires commented Oct 26, 2015

So even though code has been added to histograph/io create the indices and their mappings before indexing the document, somehow (I suspect a race condition?...) this doesn't, sometimes a document is indexed before the index is created (and thus get's automatic mapping).

I've solved this for now by creating the indices in histograph/core, before bulk indexing.

This problem needs a better solution

wires added a commit to histograph/core that referenced this issue Oct 26, 2015
Create indices before indexing documents
@wires
Copy link
Contributor Author

wires commented Oct 26, 2015

Actually, we now have this index creation in two places (here and io), needs to be removed

@wires wires reopened this Oct 26, 2015
wires added a commit to histograph/io that referenced this issue Oct 26, 2015
@wires
Copy link
Contributor Author

wires commented Oct 26, 2015

I think this duplicate index creation is what is causing errors on AWS now

GRAPHMALIZER => 108 docs, 672.628264ms
Failed creating index
{ [Error: Bad Request] status: '400', message: 'Bad Request' } 'Bad Request'
Created all indices []
ES => 56 indexed, took 66ms, errors: false

@hpfast
Copy link

hpfast commented Nov 23, 2015

I think this is affecting me too. Is solving this a question of removing it in one or the other place?

@wires
Copy link
Contributor Author

wires commented Nov 23, 2015

no, it is a complicated race condition that is not easily fixed without changing how the API/IO/redis work. In the past we used the default-mapping file (described somewhere in the installation manual) that you place in the ES config directory. On AWS this is not possible so we changed the software to create the index. The io project manages the creation and deletion of datasets, which involves creating an elasticsearch index, but when PITs/relations are pushed to the dataset io places them on the redis queue and they end up in the Graphmalizer, which then indices them into ES and this implicitly creates an index. When the indices are already there you might see a problem in the console, but it shouldn't cause a problem.

@hpfast
Copy link

hpfast commented Nov 23, 2015

When the indices are already there you might see a problem in the
console, but it shouldn't cause a problem.

ok, yeah, I saw that Graphmalizer was still proceeding. Maybe a false alarm
for me.

On Mon, Nov 23, 2015 at 2:28 PM, Jelle Herold notifications@github.com
wrote:

no, it is a complicated race condition that is not easily fixed without
changing how the API/IO/redis work. In the past we used the default-mapping
file (described somewhere in the installation manual) that you place in the
ES config directory. On AWS this is not possible so we changed the software
to create the index. The io project manages the creation and deletion of
datasets, which involves creating an elasticsearch index, but when
PITs/relations are pushed to the dataset io places them on the redis
queue and they end up in the Graphmalizer, which then indices them into ES
and this implicitly creates an index. When the indices are already there
you might see a problem in the console, but it shouldn't cause a problem.


Reply to this email directly or view it on GitHub
#72 (comment).

@wires
Copy link
Contributor Author

wires commented Nov 23, 2015

On 23 Nov 2015, at 14:47, Hans Fast notifications@github.com wrote:

When the indices are already there you might see a problem in the
console, but it shouldn't cause a problem.

ok, yeah, I saw that Graphmalizer was still proceeding. Maybe a false alarm
for me.

exactly, it’s probably a false alarm :^)

sorry for the confusion

@sbocconi
Copy link

Is this still an issue or is it resolved?

@bertspaan
Copy link
Member

Still an issue.

@sbocconi
Copy link

We are running our own instance of ES, so the problem of the default mapping is not applicable anymore. As far as I can see there is no problem in creating the indexes

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

5 participants