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

Update modules registry #3

Closed
wants to merge 711 commits into from
Closed

Conversation

shachlanAmazon
Copy link
Owner

No description provided.

barshaul and others added 30 commits September 3, 2023 09:38
Removed the FFI client from the Python wrapper
Changed pip to run with 'python -m pip'
Update cargo dependencies' github actions' versions.
This change adds to each wrapper environmental variables with an
appropriate library name and version.
Write CLIENT SETINFO data for each client.
Reduce duplication of code.
barshaul and others added 28 commits November 16, 2023 15:43
* 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.
Signed-off-by: Yury-Fridlyand <[email protected]>
Signed-off-by: Yury-Fridlyand <[email protected]>
Signed-off-by: Yury-Fridlyand <[email protected]>
Signed-off-by: Yury-Fridlyand <[email protected]>
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
Signed-off-by: Yury-Fridlyand <[email protected]>
Signed-off-by: Yury-Fridlyand <[email protected]>
Signed-off-by: Yury-Fridlyand <[email protected]>
Signed-off-by: Yury-Fridlyand <[email protected]>
Added redis installation guide to node README
Signed-off-by: Yury-Fridlyand <[email protected]>
* 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]>
…benchmarking

Add java benchmarking application. No test clients yet.
* Auxiliary changes for java client benchmarking.

Signed-off-by: Yury-Fridlyand <[email protected]>
@shachlanAmazon shachlanAmazon deleted the update-modules-registry branch December 7, 2023 14:12
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.

10 participants