Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

treemanager: Expose read-only mode and fast reads #25

Merged
merged 1 commit into from
Dec 23, 2022
Merged

treemanager: Expose read-only mode and fast reads #25

merged 1 commit into from
Dec 23, 2022

Conversation

JAORMX
Copy link
Contributor

@JAORMX JAORMX commented Dec 23, 2022

This exposes the driver's read-only mode and fast reads options to the
tree manager server. When used, this will run the tree manager in a mode
that disallows any write operations. If fast reads are used, it will
attempt to use the fast read or inconsistent mode. In CockroachDB this
translates to using follower reads to allow an instance to fetch data
from a follower instead of relying on the leader.

Signed-off-by: Juan Antonio Osorio juan.osoriorobles@eu.equinix.com

This exposes the driver's read-only mode and fast reads options to the
tree manager server. When used, this will run the tree manager in a mode
that disallows any write operations. If fast reads are used, it will
attempt to use the fast read or inconsistent mode. In CockroachDB this
translates to using follower reads to allow an instance to fetch data
from a follower instead of relying on the leader.

Signed-off-by: Juan Antonio Osorio <juan.osoriorobles@eu.equinix.com>
@codecov
Copy link

codecov bot commented Dec 23, 2022

Codecov Report

Merging #25 (b560c46) into main (fd49647) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   93.33%   93.33%           
=======================================
  Files           3        3           
  Lines         225      225           
=======================================
  Hits          210      210           
  Misses         11       11           
  Partials        4        4           
Flag Coverage Δ
unittests 93.33% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@JAORMX JAORMX merged commit a8a2805 into main Dec 23, 2022
@JAORMX JAORMX deleted the readonly branch December 23, 2022 07:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant