Skip to content

Conversation

@dokshina
Copy link
Contributor

@dokshina dokshina commented Mar 16, 2021

Added mode, prefer_replica and balance options for read operations
(get, select, pairs). According to this parameters one of vshard
calls (callrw, callro, callbro, callre, callbre) is selected.

I didn't forget about

  • Tests
  • Changelog
  • Documentation

Closes #124

Copy link
Contributor

@olegrok olegrok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! Thanks for your patch. I left several thoughts.

@dokshina dokshina force-pushed the allow-to-specify-read-operations-strategy branch from 5a399e9 to d5cf7f2 Compare March 16, 2021 17:59
@dokshina dokshina force-pushed the allow-to-specify-read-operations-strategy branch from d5cf7f2 to f8e573b Compare March 16, 2021 18:47
Added `mode`, `prefer_replica` and `balance` options for read operations
(`get`, `select`, `pairs`). According to this parameters one of vshard
calls (`callrw`, `callro`, `callbro`, `callre`, `callbre`) is selected.
@dokshina dokshina force-pushed the allow-to-specify-read-operations-strategy branch from f8e573b to 393e9d9 Compare March 17, 2021 06:59
@dokshina dokshina merged commit 6a1200a into master Mar 17, 2021
@dokshina dokshina deleted the allow-to-specify-read-operations-strategy branch March 17, 2021 08:52
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.

[4pt] Add an option to choose callro/callre/callbro/callbre on read requests

2 participants