Skip to content

Conversation

@jeffreylovitz
Copy link
Contributor

No description provided.

jeffreylovitz and others added 5 commits March 15, 2021 16:55
* Update the bulk updater to execute on a thread

* Bulk loader endpoint locks for minimal time

* TODOs

* Use a separate thread pool for bulk operations

* Update test_thread_pools.cpp

* refactor bulk-insert

* Fix PR problems

* count number of pings during bulk-insert, only create graph context on BEGIN token

Co-authored-by: swilly22 <roi@redislabs.com>
Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
(cherry picked from commit 2d43f9d)
* set score to 1 for each document

* test fulltext search scoring

* Update proc_fulltext_query.c

* Add documentation

Co-authored-by: Jeffrey Lovitz <jeffrey.lovitz@gmail.com>
(cherry picked from commit bd1fdca)
* Updated benchmark requirements version

* Update requirements.txt

(cherry picked from commit db080d4)
* Add run-time configuration for default query timeouts

* Timeout for write queries that haven't committed changes

* define TIMEOUT_NO_TIMEOUT

* Refactor timeout logic

* Address PR comments

* Do not use timeouts for write queries

Co-authored-by: swilly22 <roi@redislabs.com>
Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
(cherry picked from commit 964b268)
@jeffreylovitz jeffreylovitz requested a review from swilly22 March 15, 2021 21:00
@lgtm-com
Copy link

lgtm-com bot commented Mar 15, 2021

This pull request fixes 1 alert when merging c0f997a into aa9448d - view on LGTM.com

fixed alerts:

  • 1 for Except block handles 'BaseException'

@lgtm-com
Copy link

lgtm-com bot commented Mar 16, 2021

This pull request fixes 1 alert when merging f3209af into aa9448d - view on LGTM.com

fixed alerts:

  • 1 for Except block handles 'BaseException'

@swilly22 swilly22 merged commit bbb5d7e into 2.4 Mar 17, 2021
@swilly22 swilly22 deleted the 2-4-2-upversion branch March 17, 2021 06:15
@chayim chayim mentioned this pull request Oct 13, 2021
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.

4 participants