Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improves multiple threads parsing efficiency #57

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

qingzhongli
Copy link

Multiple threads concurrently parsing XML files causes most to be blocked. This patch changes the synchronized map to threadlocal map, which improves multiple threads parsing efficiency.

@codecov-io
Copy link

codecov-io commented Dec 10, 2018

Codecov Report

Merging #57 into master will increase coverage by <.01%.
The diff coverage is 90%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #57      +/-   ##
============================================
+ Coverage     39.93%   39.93%   +<.01%     
- Complexity     1493     1494       +1     
============================================
  Files           145      145              
  Lines          9490     9487       -3     
  Branches       1192     1192              
============================================
- Hits           3790     3789       -1     
+ Misses         5320     5319       -1     
+ Partials        380      379       -1
Impacted Files Coverage Δ Complexity Δ
src/main/java/org/dom4j/tree/QNameCache.java 74.24% <90%> (+0.32%) 16 <4> (ø) ⬇️
src/main/java/org/dom4j/tree/NamespaceCache.java 90.69% <0%> (+2.32%) 13% <0%> (+1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8073066...76892de. Read the comment docs.

@FilipJirsak FilipJirsak added this to the 2.2.0 milestone Jan 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants