Add CI tests for benchmarks. #2

shachlanAmazon · 2023-12-05T19:03:50Z

Allow benchmarks to run in a minimal fashion, that shouldn't take a long while.
Add CI task for each language, so that benchmarks will run for it.
Add relevant file extensions for changes that should trigger CI for each language.

Python - config rewrite command

Merging updates from redis-rs

Throw concrete error types from node client.

Fixing Python's INFO parser to support parsing Redis 7.2 INFO output

JS might perform a lot of tests when checking if an object is truthy.

This saves the cost of allocating a new error on every promise. Apparently this has a significant perf cost.

Fix node's performance degradation.

Removed the FFI client from the Python wrapper

update the doc comments to TSDoc

Updates from redis-rs

Fixed config rewrite tests

…ssmethod functions

…stems and Python versions.

Create bug-report.yml

…d from source

…empl Create feature-request.yml

* Align indentation for java .yml files. All other .yml files use this indentation. * Run Java's CI only on branch pushes to main. We don't need to run CI on every push to every development branch.

These timeouts carry a significant performance penalty. Their benefit is more precise timeouts to the user, but we're willing to be off by a bit in order to improve perf.

This means that if you pass the `--stripInternal` flag to `tsc`, these functions won't be emitted in the typescript declaration files, and we won't expose the protobuf types. temp

Added redis installation guide to node README

https://github.com/redis-rs/redis-rs/blob/882ecf9c5da4a6cecaea059194be5a71f86befcf/redis/Cargo.toml#L111

…e mode

* Remove csharp socket client. * C#: Use our client, not the redis-rs multiplexer. * Use generics in MessageContainer. This allows us to move some of the message logic to MessageContainer. * Fix benchmark program. * Message: Implement awaiter. `Message` now implements GetAwaiter, which allows the TPL to await on `Message`s, which is much more efficient, and removes the need for `ValueTask`s. As a part of the change, now `Message`s report to their `MessageContainer` when they complete an action, and can be made available again. Additionally, `Message`s now can return `null` regardless of their generic type - this is done in order to remove a warning, since the code can't ensure that the returned value isn't `null` without additional runtime checks, and `null` values are legal in our flows. * re-enable csharp github actions. * Fix compilation. * PR comments. * C#: Rename connection to client. --------- Co-authored-by: nihohit <[email protected]>

barshaul and others added 30 commits August 31, 2023 07:03

Fixing info parser to support parsing Redis 7.2 output

273ee22

Merging updates from redis-rs

9c1270d

Python - config rewrite command (#380)

f50aec7

Python - config rewrite command

Python - refactor config reset stat function (#391)

dd3d915

Merge pull request #394 from barshaul/check_babushka_tls

82175f0

Merging updates from redis-rs

Merge pull request #357 from shachlanAmazon/node-errors

15fa004

Throw concrete error types from node client.

Extended redis version matrix to redis 6.2 & 7.2

02373af

Merge pull request #393 from barshaul/fix_info_parser

ad51413

Fixing Python's INFO parser to support parsing Redis 7.2 INFO output

Use simpler existence checks.

65fab05

JS might perform a lot of tests when checking if an object is truthy.

Use contstant timeout error.

3a8014f

This saves the cost of allocating a new error on every promise. Apparently this has a significant perf cost.

Merge pull request #396 from shachlanAmazon/reject-messages

cb525c9

Fix node's performance degradation.

Removed the FFI client from the Python wrapper

4aba34f

Merge pull request #400 from barshaul/remove_py_ffi

87a621c

Removed the FFI client from the Python wrapper

TSDoc linter added & update the doc comments

2869447

fix tsdoc-linter

125d67b

check the linter

c555c9d

update to error in linter

1808765

fix the TSdoc comment in transaction

7b7f2ab

Merge pull request #392 from barshaul/node/updateDoc

69ce79a

update the doc comments to TSDoc

added config rewrite command in node

9392b39

Updates from redis-rs

857a0c0

Fixed config rewrite tests

a0f7303

Created protobuf codec for Python

64e3758

Merge pull request #410 from barshaul/update_redisrs

42edf14

Updates from redis-rs

Merge pull request #411 from barshaul/fix_rewrite_test

d166b05

Fixed config rewrite tests

added incr and incrBy commands in node.

10315a2

Changed Python Logger set_logger_config and log functions to be a cla…

1b71590

…ssmethod functions

Update cargo dependencies.

6dae44c

Update Github actions versions.

fe470df

Changed pip to run with 'python -m pip'

77da9be

barshaul and others added 29 commits November 14, 2023 11:07

Fixed main and Python README based on the review notes

1668ad5

Python: Minor changese to add compatibility with various operating sy…

fad22c1

…stems and Python versions.

Python: added MacOS and amazonlinux build tests to github actions CI

3b4922c

Create bug-report.yml

a78e01c

Merge pull request #616 from aws/alexander-shabanov-bug-report-template

8056ce0

Create bug-report.yml

Node: add routing option to exec command

90c43fd

Python: add --strip flag to remove debug symbols from the readme buil…

985511b

…d from source

Added MacOS and AmazonLinux build tests to Node's CI

883f3b6

Node: Updated README file

ab29a72

Create feature-request.yml

7b31ce0

Merge pull request #620 from aws/alexander-shabanov-feature-request-t…

9ebc3bc

…empl Create feature-request.yml

Python: Added DEVELOPER.md file

b3c7e9c

Remove client_creation_timeout configuration

a93d999

Run Java's CI only on branch pushes to main. (#624)

199507f

* Align indentation for java .yml files. All other .yml files use this indentation. * Run Java's CI only on branch pushes to main. We don't need to run CI on every push to every development branch.

Fix cargo.toml files

3659b67

Node: Added DEVELOPER.md file

f6ac486

Node: Remove timeout on requests.

1836f4e

These timeouts carry a significant performance penalty. Their benefit is more precise timeouts to the user, but we're willing to be off by a bit in order to improve perf.

Fixed CONTRIBUTING, expended bug-repot.yml

42dc1b1

Node: mark protected functions as internal.

f6d2050

This means that if you pass the `--stripInternal` flag to `tsc`, these functions won't be emitted in the typescript declaration files, and we won't expose the protobuf types. temp

Fix linter comment.

bf75a20

Update README.md

c9fcd99

Added redis installation guide to node README

Remove usage of deprecated "tls" feature.

5f6beb4

https://github.com/redis-rs/redis-rs/blob/882ecf9c5da4a6cecaea059194be5a71f86befcf/redis/Cargo.toml#L111

Java update readme (#636)

b6f1d22

Extand cluster_manager.py script to support replication on standealon…

6668c63

…e mode

Fixed yaml format

29bc89b

Allow running minimal number of ops in benchmarks.

3d2868b

Add CI action to check that benchmarks work.

e5ca5b2

Run CI on more file changes.

ae55087

shachlanAmazon closed this Dec 5, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add CI tests for benchmarks. #2

Add CI tests for benchmarks. #2

shachlanAmazon commented Dec 5, 2023

Add CI tests for benchmarks. #2

Add CI tests for benchmarks. #2

Conversation

shachlanAmazon commented Dec 5, 2023