Skip to content

Conversation

@jeffreylovitz
Copy link
Contributor

No description provided.

swilly22 and others added 25 commits March 10, 2021 09:56
* Added local and remote benchmark definition and automation

* [fix] Fixes per PR review. Added option to specify benchmark via BENCHMARH=<benchmark name>. Updated benchmark template

Co-authored-by: filipecosta90 <filipecosta.90@gmail.com>
(cherry picked from commit f6f1ab2)
…cy KPI (#1577)

Given we're still experimenting with the benchmarks CI KPI validation, this PR increases the `OverallClientLatencies.Total.q50` to be lower than 2.0 ( before was 1.5 ) so that we can collect further data and adjust afterwards...

(cherry picked from commit 611a0f0)
When pulling container image tagged as `latest` or `edge` I sometimes
don't know which version I'm running, and it would be much faster to
find out if the information was displayed at startup. This patch logs
this information.

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
(cherry picked from commit fe2e7ce)
* Fix flawed assertion in index deletion logic

* Reduce KPI for updates_baseline benchmark

* Address PR comments

* Address PR comments

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
(cherry picked from commit 6bad20a)
* Always report run-time errors as the sole reply

* Update test_timeout.py

Co-authored-by: Roi Lipman <swilly22@users.noreply.github.com>
(cherry picked from commit c9ba776)
(cherry picked from commit 12ef8ac)
Multi-platform build

(cherry picked from commit 26ace7a)
…mproved benchmark specification file (#1597)

(cherry picked from commit ebea927)
* Added readies submodule

* fixes 1

(cherry picked from commit efbfeaf)
(cherry picked from commit 7ea1c43)
* 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)
(cherry picked from commit 0c10130)
* 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 10, 2021 15:46
@swilly22 swilly22 merged commit e611266 into 2.2 Mar 15, 2021
@swilly22 swilly22 deleted the 2-2-16-upversion branch March 15, 2021 07:13
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.

6 participants