olabusayoT opened a new pull request, #1712: URL: https://github.com/apache/daffodil/pull/1712
NextElementResolver's OnlyOnePossibilityForNextElement and SeveralPossibilitiesForNextElement both call NS(namespace) on every element resolution when the infoset source has namespaces. NS.apply(String) previously ran URI.create plus a ReentrantReadWriteLock-guarded WeakHashMap lookup on every call, even for a namespace string seen many times before - real per-element overhead on unparse's next-element-resolution hot path. A ConcurrentHashMap keyed by the raw string now short-circuits repeats before they ever reach URI.create or the lock. DAFFODIL-3092 -- 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 the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
