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

[Task] Rename the Modules from Transaction to Pipeline #3030

Open
2 tasks
BoazBD opened this issue Jan 30, 2025 · 1 comment
Open
2 tasks

[Task] Rename the Modules from Transaction to Pipeline #3030

BoazBD opened this issue Jan 30, 2025 · 1 comment
Assignees
Labels
All Wrappers issue Issue relevant to all wrrappers. Release blocker Can't release without.
Milestone

Comments

@BoazBD
Copy link
Collaborator

BoazBD commented Jan 30, 2025

Description

In order to add pipeline support, we should rename the modules from transaction to pipeline.
For example, change the name of the JSON module from json_transaction.py --> json_pipeline.py
We should do that to not confuse clients, after adding the pipeline support.
The behavior of pipelines and the transactions is very similar, the modules should be identical to them.
The renaming is due to the fact that pipelines are broader, making "pipeline" a more accurate term.

This should break any changes, since the JSON and VSS modules are not yet released.

Checklist

  • JSON Module
  • VSS Module

Additional Notes

No response

@shohamazon shohamazon added this to the 1.3 milestone Feb 2, 2025
@shohamazon shohamazon added Release blocker Can't release without. All Wrappers issue Issue relevant to all wrrappers. labels Feb 2, 2025
@shohamazon shohamazon moved this to In Progress in Valkey-GLIDE - internal Feb 3, 2025
@shohamazon
Copy link
Collaborator

rename to Batching

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
All Wrappers issue Issue relevant to all wrrappers. Release blocker Can't release without.
Projects
Status: In Progress
Development

No branches or pull requests

2 participants