Skip to content

RATIS-2397. Add trace support for Log Appender#1443

Open
taklwu wants to merge 1 commit intoapache:masterfrom
taklwu:RATIS-2397
Open

RATIS-2397. Add trace support for Log Appender#1443
taklwu wants to merge 1 commit intoapache:masterfrom
taklwu:RATIS-2397

Conversation

@taklwu
Copy link
Copy Markdown
Contributor

@taklwu taklwu commented Apr 30, 2026

What changes were proposed in this pull request?

Adding tracing support for AppendEntries / AppendEntriesAsync

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/RATIS-2397

How was this patch tested?

  1. added unit tests
  2. build with RATIS-2398. Add opentelemetry-javaagent to ratis-examples and assembly #1428 and test locally with Jaegar UI to see the spans have been capture.
Screenshot 2026-04-29 at 5 04 41 PM Screenshot 2026-04-29 at 4 54 36 PM

@taklwu taklwu changed the title RATIS-2397 Add trace support for Log Appender RATIS-2397. Add trace support for Log Appender Apr 30, 2026
@taklwu
Copy link
Copy Markdown
Contributor Author

taklwu commented Apr 30, 2026

@szetszwo so , I found the checkstyle is complaining about max lines in a single java class, does it mean we need to split this file into separate classes ?

ratis-server/src/main/java/org/apache/ratis/server/impl/RaftServerImpl.java
 1: File length is 2,002 lines (max allowed is 2,000).

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.

1 participant