Re: [PR] Changed simple synchronization to ReentrantReadWriteLock [tomcat]

2024-07-03 Thread via GitHub
n828cl commented on PR #712: URL: https://github.com/apache/tomcat/pull/712#issuecomment-2208075153 Tomcat 8.5 reached end-of-life in March, so there won't be any more updates. You should update to a supported level at your earliest convenience. -- This is an automated message from the Ap

Re: [PR] Changed simple synchronization to ReentrantReadWriteLock [tomcat]

2024-07-03 Thread via GitHub
wapkch commented on PR #712: URL: https://github.com/apache/tomcat/pull/712#issuecomment-2207881666 @ChristopherSchultz Could this fix be ported to 8.5.x? Cause the latest 8.5.100 has the same problem. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] Changed simple synchronization to ReentrantReadWriteLock [tomcat]

2024-04-02 Thread via GitHub
ChristopherSchultz commented on PR #712: URL: https://github.com/apache/tomcat/pull/712#issuecomment-2032266991 Thank you for this PR, though I was already working on a patch and the unit tests pass, so I'm going to commit my patch instead of yours. I will credit you for the idea in the cha

Re: [PR] Changed simple synchronization to ReentrantReadWriteLock [tomcat]

2024-04-02 Thread via GitHub
ChristopherSchultz closed pull request #712: Changed simple synchronization to ReentrantReadWriteLock URL: https://github.com/apache/tomcat/pull/712 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to t

[PR] Changed simple synchronization to ReentrantReadWriteLock [tomcat]

2024-04-02 Thread via GitHub
LupusMKW opened a new pull request, #712: URL: https://github.com/apache/tomcat/pull/712 Due to https://github.com/apache/tomcat/commit/4f33be682fda02a616baa0fd9b4965d248cfe1c1 our application does no longer work. The commit improved the situation with respect to concurrent mutabili