Github user ajs6f commented on a diff in the pull request:
https://github.com/apache/jena/pull/294#discussion_r146827609
--- Diff:
jena-arq/src/main/java/org/apache/jena/riot/adapters/AdapterFileManager.java ---
@@ -285,6 +286,12 @@ protected Model readModelWorker(Model model, String
filenameOrURI, String baseUR
if ( baseURI == null )
baseURI = SysRIOT.chooseBaseIRI(filenameOrURI) ;
try(TypedInputStream in =
streamManager.openNoMapOrNull(mappedURI)) {
+ if ( in == null )
+ {
+ if ( log.isDebugEnabled() )
+ log.debug("Failed to locate '"+mappedURI+"'") ;
--- End diff --
As I understand it, `log.debug("Failed to locate '{}'", mappedURI) `
[avoids the need](https://www.slf4j.org/faq.html#logging_performance) to
explicitly check `isDebugEnabled()`.
---
