[jira] [Created] (THRIFT-5897) Use read lock for listener access in TServerSocket.Accept()

["Atharva Chiplunkar (Jira)" <ji...@apache.org>]

Atharva Chiplunkar created THRIFT-5897:
------------------------------------------

             Summary: Use read lock for listener access in TServerSocket.Accept()
                 Key: THRIFT-5897
                 URL: https://issues.apache.org/jira/browse/THRIFT-5897
             Project: Thrift
          Issue Type: Bug
          Components: Go - Library
            Reporter: Atharva Chiplunkar
            Assignee: Atharva Chiplunkar


Currently, TServerSocket.Accept() uses two separate lock acquisitions:
1. RLock for reading 'interrupted' field
2. write lock for reading 'listener' field

Since both fields are only being read (not modified), both can be protected 
by a single read lock (RLock). This optimization:
- Reduces lock contention by eliminating unnecessary write lock
- Allows multiple Accept() calls to proceed concurrently

Change:
- Combine both field reads under a single RLock/RUnlock pair
- Remove unnecessary write lock (Lock/Unlock)

This is a performance optimization with no functional change.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)