futil-js / contexture-core

Contexture DSL Processor
MIT License
21 stars 1 forks source link

Revert "Fix/performance" #66

Closed daedalus28 closed 3 years ago

daedalus28 commented 3 years ago

Reverts smartprocure/contexture#63

Running all searches in parallel is not necessarily a good thing - and in fact, on an elastic cluster with a maximum number of parallel queries, can be actively harmful for overall availability and stability of the cluster. Coupled with the multisearch PR for contexture-elasticsearch, this might be a good thing - but we need to do extensive testing and likely allow a way to switch between parallel and serial on a per provider, schema, and maybe even per tree basis.

decrapifier commented 3 years ago
Warnings
:warning: The README has not been updated. Please update the README.
:warning: Branch being merged does not follow [Git Flow](http://nvie.com/posts/a-successful-git-branching-model/)
:warning: You should add at least 1 more reviewer to the PR
Messages
:book: Could not find any browser results.
:book: You reduced the total lines of code! Awesome! :+1:

Generated by :no_entry_sign: dangerJS against 7e8b907ce43c1410ba7d10d41a069d1da8d82296