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

Go: Added config folder for request_routing_config_test.go and errors folder for errors.go #2996

Merged
merged 12 commits into from
Jan 24, 2025

Conversation

niharikabhavaraju
Copy link
Collaborator

Go: Added config, errors folders

@niharikabhavaraju niharikabhavaraju requested a review from a team as a code owner January 23, 2025 05:26
@niharikabhavaraju niharikabhavaraju changed the title Go: Added config, errors folders Go: Added config folder for request_routing_config_test.go and errors folder for errors.go Jan 23, 2025
@janhavigupta007 janhavigupta007 added the go golang wrapper label Jan 23, 2025
@niharikabhavaraju niharikabhavaraju self-assigned this Jan 23, 2025
@vikaskpandey vikaskpandey merged commit a1360c2 into valkey-io:main Jan 24, 2025
18 checks passed
EdricCua pushed a commit to EdricCua/valkey-glide that referenced this pull request Jan 24, 2025
… folder for errors.go (valkey-io#2996)

* Implemented Time Command

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review changes

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added Time command (cluster)

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review comments

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added route,errors folders

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed failing tests

Signed-off-by: Niharika Bhavaraju <[email protected]>

---------

Signed-off-by: Niharika Bhavaraju <[email protected]>
edlng pushed a commit to Bit-Quill/valkey-glide that referenced this pull request Jan 27, 2025
… folder for errors.go (valkey-io#2996)

* Implemented Time Command

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review changes

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added Time command (cluster)

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review comments

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added route,errors folders

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed failing tests

Signed-off-by: Niharika Bhavaraju <[email protected]>

---------

Signed-off-by: Niharika Bhavaraju <[email protected]>
Signed-off-by: Edward Liang <[email protected]>
EdricCua pushed a commit to EdricCua/valkey-glide that referenced this pull request Jan 28, 2025
… folder for errors.go (valkey-io#2996)

* Implemented Time Command

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review changes

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added Time command (cluster)

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review comments

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added route,errors folders

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed failing tests

Signed-off-by: Niharika Bhavaraju <[email protected]>

---------

Signed-off-by: Niharika Bhavaraju <[email protected]>
EdricCua pushed a commit to EdricCua/valkey-glide that referenced this pull request Jan 28, 2025
… folder for errors.go (valkey-io#2996)

* Implemented Time Command

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review changes

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added Time command (cluster)

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review comments

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added route,errors folders

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed failing tests

Signed-off-by: Niharika Bhavaraju <[email protected]>

---------

Signed-off-by: Niharika Bhavaraju <[email protected]>
Signed-off-by: EdricCua <[email protected]>
vikaskpandey pushed a commit that referenced this pull request Jan 31, 2025
* Implement DBSize Server Management Cluster

Signed-off-by: EdricCua <[email protected]>

* Fix docs

Signed-off-by: EdricCua <[email protected]>

* Fix docs

Signed-off-by: EdricCua <[email protected]>

* ORT- Add resolutions for hybrid-node-tests and bump versions (#2987)

* add resolutions for hybrid-node-tests and bump ORT versions

Signed-off-by: BoazBD <[email protected]>

* format

Signed-off-by: BoazBD <[email protected]>

---------

Signed-off-by: BoazBD <[email protected]>
Signed-off-by: EdricCua <[email protected]>

* Go: Implement Server Management Command DBSize (#2991)

* Implement DBSize Command

Signed-off-by: EdricCua <[email protected]>

* GO: Implement Copy Command (#2980)

* Implement Copy Command

Signed-off-by: EdricCua <[email protected]>

* Moved documentation comments to base_client file and made changes to GlideClient for pkgsite to work (#2981)

* moved docs from interfaces to base client

Signed-off-by: Edward Liang <[email protected]>
Signed-off-by: EdricCua <[email protected]>

* quickfix of changing glideClient to become GlideClient (#3000)

Signed-off-by: Edward Liang <[email protected]>
Signed-off-by: EdricCua <[email protected]>

* Go: Add command XRange and XRevRange (#2989)

* Go: Add command XRange and XRevRange

Signed-off-by: TJ Zhang <[email protected]>
Signed-off-by: EdricCua <[email protected]>

* Go: Added config folder for request_routing_config_test.go and errors folder for errors.go (#2996)

* Implemented Time Command

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review changes

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added Time command (cluster)

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review comments

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added route,errors folders

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed failing tests

Signed-off-by: Niharika Bhavaraju <[email protected]>

---------

Signed-off-by: Niharika Bhavaraju <[email protected]>
Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Fix merge conflict

Signed-off-by: EdricCua <[email protected]>

* Fix review comment

Signed-off-by: EdricCua <[email protected]>

* Fix docs for RouteOption

Signed-off-by: EdricCua <[email protected]>

---------

Signed-off-by: EdricCua <[email protected]>
Signed-off-by: BoazBD <[email protected]>
Signed-off-by: Edward Liang <[email protected]>
Signed-off-by: TJ Zhang <[email protected]>
Signed-off-by: Niharika Bhavaraju <[email protected]>
Co-authored-by: BoazBD <[email protected]>
Co-authored-by: Edward Liang <[email protected]>
Co-authored-by: tjzhang-BQ <[email protected]>
Co-authored-by: Niharika Bhavaraju <[email protected]>
adarovadya pushed a commit to adarovadya/glide-for-redis that referenced this pull request Feb 3, 2025
* Implement DBSize Server Management Cluster

Signed-off-by: EdricCua <[email protected]>

* Fix docs

Signed-off-by: EdricCua <[email protected]>

* Fix docs

Signed-off-by: EdricCua <[email protected]>

* ORT- Add resolutions for hybrid-node-tests and bump versions (valkey-io#2987)

* add resolutions for hybrid-node-tests and bump ORT versions

Signed-off-by: BoazBD <[email protected]>

* format

Signed-off-by: BoazBD <[email protected]>

---------

Signed-off-by: BoazBD <[email protected]>
Signed-off-by: EdricCua <[email protected]>

* Go: Implement Server Management Command DBSize (valkey-io#2991)

* Implement DBSize Command

Signed-off-by: EdricCua <[email protected]>

* GO: Implement Copy Command (valkey-io#2980)

* Implement Copy Command

Signed-off-by: EdricCua <[email protected]>

* Moved documentation comments to base_client file and made changes to GlideClient for pkgsite to work (valkey-io#2981)

* moved docs from interfaces to base client

Signed-off-by: Edward Liang <[email protected]>
Signed-off-by: EdricCua <[email protected]>

* quickfix of changing glideClient to become GlideClient (valkey-io#3000)

Signed-off-by: Edward Liang <[email protected]>
Signed-off-by: EdricCua <[email protected]>

* Go: Add command XRange and XRevRange (valkey-io#2989)

* Go: Add command XRange and XRevRange

Signed-off-by: TJ Zhang <[email protected]>
Signed-off-by: EdricCua <[email protected]>

* Go: Added config folder for request_routing_config_test.go and errors folder for errors.go (valkey-io#2996)

* Implemented Time Command

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review changes

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added Time command (cluster)

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed code review comments

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Added route,errors folders

Signed-off-by: Niharika Bhavaraju <[email protected]>

* Fixed failing tests

Signed-off-by: Niharika Bhavaraju <[email protected]>

---------

Signed-off-by: Niharika Bhavaraju <[email protected]>
Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Implement DBSize Cluster

Signed-off-by: EdricCua <[email protected]>

* Fix merge conflict

Signed-off-by: EdricCua <[email protected]>

* Fix review comment

Signed-off-by: EdricCua <[email protected]>

* Fix docs for RouteOption

Signed-off-by: EdricCua <[email protected]>

---------

Signed-off-by: EdricCua <[email protected]>
Signed-off-by: BoazBD <[email protected]>
Signed-off-by: Edward Liang <[email protected]>
Signed-off-by: TJ Zhang <[email protected]>
Signed-off-by: Niharika Bhavaraju <[email protected]>
Co-authored-by: BoazBD <[email protected]>
Co-authored-by: Edward Liang <[email protected]>
Co-authored-by: tjzhang-BQ <[email protected]>
Co-authored-by: Niharika Bhavaraju <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go golang wrapper
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants