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

Optimize log for failed to write entry #3463

Merged
merged 1 commit into from Sep 5, 2022

Conversation

AnonHxy
Copy link
Contributor

@AnonHxy AnonHxy commented Aug 31, 2022

Descriptions of the changes in this PR:

Print bookie address when writing entry failed

Motivation

Printing bookie info will be help for solving online problem

Changes

Print bookie address when writing entry failed

@AnonHxy AnonHxy closed this Aug 31, 2022
@AnonHxy AnonHxy reopened this Aug 31, 2022
Copy link
Contributor

@StevenLuMT StevenLuMT left a comment

Choose a reason for hiding this comment

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

lgtm

@hangc0276 hangc0276 added this to the 4.16.0 milestone Sep 5, 2022
@hangc0276 hangc0276 merged commit 1adc041 into apache:master Sep 5, 2022
zymap pushed a commit that referenced this pull request Sep 21, 2022
Descriptions of the changes in this PR:

Print bookie address when writing entry failed

### Motivation

Printing bookie info will be help for solving online problem
### Changes

Print bookie address when writing entry failed

(cherry picked from commit 1adc041)
hangc0276 pushed a commit to hangc0276/bookkeeper that referenced this pull request Nov 7, 2022
Descriptions of the changes in this PR:

Print bookie address when writing entry failed

### Motivation

Printing bookie info will be help for solving online problem
### Changes

Print bookie address when writing entry failed

(cherry picked from commit 1adc041)
hangc0276 pushed a commit to hangc0276/bookkeeper that referenced this pull request Nov 7, 2022
Descriptions of the changes in this PR:

Print bookie address when writing entry failed

### Motivation

Printing bookie info will be help for solving online problem
### Changes

Print bookie address when writing entry failed

(cherry picked from commit 1adc041)
nicoloboschi pushed a commit to datastax/bookkeeper that referenced this pull request Jan 11, 2023
Descriptions of the changes in this PR:

Print bookie address when writing entry failed

### Motivation

Printing bookie info will be help for solving online problem
### Changes

Print bookie address when writing entry failed

(cherry picked from commit 1adc041)
(cherry picked from commit a57f24b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants