Skip to content

RATIS-1543. Log from GrpcLogAppender is confusing.#821

Merged
codings-dan merged 1 commit intoapache:masterfrom
szetszwo:RATIS-1543
Feb 17, 2023
Merged

RATIS-1543. Log from GrpcLogAppender is confusing.#821
codings-dan merged 1 commit intoapache:masterfrom
szetszwo:RATIS-1543

Conversation

@szetszwo
Copy link
Copy Markdown
Contributor

@szetszwo szetszwo requested a review from SzyWilliam February 16, 2023 23:42
Copy link
Copy Markdown
Member

@SzyWilliam SzyWilliam left a comment

Choose a reason for hiding this comment

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

@szetszwo Thanks a lot for working on this! New logs seem pretty helpful.

Copy link
Copy Markdown
Contributor

@codings-dan codings-dan left a comment

Choose a reason for hiding this comment

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

the change looks good, +1

@codings-dan codings-dan merged commit ca9fdd2 into apache:master Feb 17, 2023
kaijchen pushed a commit to kaijchen/ratis that referenced this pull request Feb 26, 2023
szetszwo added a commit that referenced this pull request Mar 1, 2023
szetszwo added a commit that referenced this pull request Mar 5, 2023
symious pushed a commit to symious/ratis that referenced this pull request Mar 14, 2024
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