Skip to content

Conversation

kyryloholodnov
Copy link

This version is necessary for multithreaded environment. LRUMap from org.apache.commons.collections.map can hang in multithreaded production environment which is fully reproducible in high load.

@kyryloholodnov
Copy link
Author

Any update on this?

Copy link
Contributor

@douglaspalmer douglaspalmer left a comment

Choose a reason for hiding this comment

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

After updating this PR it is good.

}
});

public ConcurrentCachingParser() throws IOException {
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove throws IOException

public class ConcurrentCachingParserTest extends ParserTest {

@Before
public void initParser() throws Exception {
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove throws Exception

<dependency>
<groupId>com.google.guava</groupId>
<artifactId>guava</artifactId>
<version>23.0</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

There's newer versions of this. Lets update to 31.1-jre

@douglaspalmer
Copy link
Contributor

@bpossolo Is there any chance of getting this merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants