Skip to content

Issue #4130: [doc][site] split performance in two parts and add sticky reads part. - #4131

Merged
hangc0276 merged 1 commit into
apache:masterfrom
KannarFr:addStickyReadsBulletPoint
Jan 2, 2024
Merged

hangc0276 merged 1 commit into
apache:masterfrom
KannarFr:addStickyReadsBulletPoint

Conversation

@KannarFr

Copy link
Copy Markdown
Contributor

Descriptions of the changes in this PR:

Motivation

After the issue on disabled sticky reads, I'm adding this to improve documentation about performance on reading.

Changes

Master Issue: #4130

@KannarFr KannarFr changed the title <Issue #4130>: [doc][site] split performance in two parts and add sticky reads part. Issue #4130: [doc][site] split performance in two parts and add sticky reads part. Nov 15, 2023

#### Sticky reads

BookKeeper sticky reads enables bookie node to read entries efficiently. The sticky reads are only available when ensemble (E) size is equal to quorum write (Qw).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@hangc0276 Can you verify me? You are the expert here

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@hangc0276

Copy link
Copy Markdown
Contributor

@KannarFr Please rebase the master, thanks.

@KannarFr
KannarFr force-pushed the addStickyReadsBulletPoint branch from 4e05cd1 to 5f391b5 Compare December 8, 2023 11:59
@KannarFr

KannarFr commented Dec 8, 2023

Copy link
Copy Markdown
Contributor Author

@hangc0276 done.

@hangc0276
hangc0276 merged commit d189c20 into apache:master Jan 2, 2024
Ghatage pushed a commit to sijie/bookkeeper that referenced this pull request Jul 12, 2024
Descriptions of the changes in this PR:


### Motivation

After the issue on disabled sticky reads, I'm adding this to improve documentation about performance on reading.

### Changes

Master Issue: apache#4130
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.

3 participants