[ZOOKEEPER-4460] QuorumPeer overrides Thread.getId with different semantics

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.8.1, 3.7.2
• Component/s: server
• Labels: pull-request-available
• Assignee: Chris Nauroth
• Reporter: Alan Bateman
• Created: 28/Jan/22 08:13
• Updated: 30/Jan/23 07:57
• Resolved: 03/Nov/22 06:06

Description

In OpenJDK, Project Loom has significantly re-implemented java.lang.Thread. One part of this is using some of the bits in the thread identifier for non-exposed purposes. Sadly, Thread::getId is not final and it's possible that sub-classes of Thread have overridden getId to have different semantics. The JDK can defend against this but there may be 3rd party libraries that make use of Thread::getId. A corpus search of Maven central found only one class: org.apache.zookeeper.server.quorum.QuorumPeer. Does this project know why getId has been overridden to return something that is not the thread identifier?

Issue Links

• links to: GitHub Pull Request #1942