[MediaWiki-commits] [Gerrit] wikidata...rdf[master]: Make all logging use slf4j since this is what Blazegraph is ...
jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/349128 ) Change subject: Make all logging use slf4j since this is what Blazegraph is using. .. Make all logging use slf4j since this is what Blazegraph is using. Also fix some style warnings. Change-Id: I345fddc126ed054ed9ebc00b3266184747a9ee41 --- M blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/literal/AbstractMultiTypeExtension.java M blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/uri/WikibaseStyleStatementInlineUriHandler.java M blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/EmptyLabelServiceOptimizer.java M blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/LabelService.java M blazegraph/src/test/java/org/wikidata/query/rdf/blazegraph/label/LabelServiceUnitTest.java 5 files changed, 16 insertions(+), 16 deletions(-) Approvals: Smalyshev: Looks good to me, approved jenkins-bot: Verified diff --git a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/literal/AbstractMultiTypeExtension.java b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/literal/AbstractMultiTypeExtension.java index eaf51f4..018e5e9 100644 --- a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/literal/AbstractMultiTypeExtension.java +++ b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/literal/AbstractMultiTypeExtension.java @@ -9,10 +9,11 @@ import java.util.Map; import java.util.Set; -import org.apache.log4j.Logger; import org.openrdf.model.Literal; import org.openrdf.model.URI; import org.openrdf.model.Value; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; import com.bigdata.rdf.internal.IDatatypeURIResolver; import com.bigdata.rdf.internal.IExtension; @@ -32,7 +33,7 @@ *roughly by Blazegraph - lots of rawtypes */ public abstract class AbstractMultiTypeExtension implements IExtension { -private static final Logger log = Logger.getLogger(WikibaseDateExtension.class); +private static final Logger log = LoggerFactory.getLogger(WikibaseDateExtension.class); /** * IV to type map as resolved against resolver provided on construction. diff --git a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/uri/WikibaseStyleStatementInlineUriHandler.java b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/uri/WikibaseStyleStatementInlineUriHandler.java index 1c158a4..1c1c027 100644 --- a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/uri/WikibaseStyleStatementInlineUriHandler.java +++ b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/uri/WikibaseStyleStatementInlineUriHandler.java @@ -4,7 +4,8 @@ import java.util.Locale; import java.util.UUID; -import org.apache.log4j.Logger; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; import com.bigdata.rdf.internal.InlineURIHandler; import com.bigdata.rdf.internal.impl.literal.AbstractLiteralIV; @@ -29,7 +30,7 @@ * @deprecated */ public class WikibaseStyleStatementInlineUriHandler extends InlineURIHandler { -private static final Logger log = Logger.getLogger(WikibaseStyleStatementInlineUriHandler.class); +private static final Logger log = LoggerFactory.getLogger(WikibaseStyleStatementInlineUriHandler.class); public WikibaseStyleStatementInlineUriHandler(String namespace) { super(namespace); @@ -99,7 +100,7 @@ i = i.shiftLeft(Long.SIZE).or(unsigned(u.getLeastSignificantBits())); return new XSDIntegerIV(i); } catch (IllegalArgumentException e) { - Logger.getLogger(WikibaseStyleStatementInlineUriHandler.class).warn("tmp", e); +log.warn("tmp", e); return null; } } diff --git a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/EmptyLabelServiceOptimizer.java b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/EmptyLabelServiceOptimizer.java index 44cb389..e5922bd 100644 --- a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/EmptyLabelServiceOptimizer.java +++ b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/EmptyLabelServiceOptimizer.java @@ -2,11 +2,12 @@ import java.util.List; -import org.apache.log4j.Logger; import org.openrdf.model.URI; import org.openrdf.model.impl.URIImpl; import org.openrdf.model.vocabulary.RDFS; import org.openrdf.model.vocabulary.SKOS; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; import org.wikidata.query.rdf.common.uri.Ontology; import org.wikidata.query.rdf.common.uri.SchemaDotOrg; @@ -36,7 +37,7 @@ */ @SuppressWarnings("rawtypes") public class EmptyLabelServiceOptimizer extends AbstractJoinGroupOptimizer { -private static final Logger log = Logger.getLogger(EmptyLabelServiceOptimizer.class); +
[MediaWiki-commits] [Gerrit] wikidata...rdf[master]: Make all logging use slf4j since this is what Blazegraph is ...
Smalyshev has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/349128 ) Change subject: Make all logging use slf4j since this is what Blazegraph is using. .. Make all logging use slf4j since this is what Blazegraph is using. Also fix some style warnings. Change-Id: I345fddc126ed054ed9ebc00b3266184747a9ee41 --- M blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/literal/AbstractMultiTypeExtension.java M blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/uri/WikibaseStyleStatementInlineUriHandler.java M blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/EmptyLabelServiceOptimizer.java M blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/LabelService.java M blazegraph/src/test/java/org/wikidata/query/rdf/blazegraph/label/LabelServiceUnitTest.java 5 files changed, 16 insertions(+), 16 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/wikidata/query/rdf refs/changes/28/349128/1 diff --git a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/literal/AbstractMultiTypeExtension.java b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/literal/AbstractMultiTypeExtension.java index eaf51f4..018e5e9 100644 --- a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/literal/AbstractMultiTypeExtension.java +++ b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/literal/AbstractMultiTypeExtension.java @@ -9,10 +9,11 @@ import java.util.Map; import java.util.Set; -import org.apache.log4j.Logger; import org.openrdf.model.Literal; import org.openrdf.model.URI; import org.openrdf.model.Value; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; import com.bigdata.rdf.internal.IDatatypeURIResolver; import com.bigdata.rdf.internal.IExtension; @@ -32,7 +33,7 @@ *roughly by Blazegraph - lots of rawtypes */ public abstract class AbstractMultiTypeExtension implements IExtension { -private static final Logger log = Logger.getLogger(WikibaseDateExtension.class); +private static final Logger log = LoggerFactory.getLogger(WikibaseDateExtension.class); /** * IV to type map as resolved against resolver provided on construction. diff --git a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/uri/WikibaseStyleStatementInlineUriHandler.java b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/uri/WikibaseStyleStatementInlineUriHandler.java index 1c158a4..1c1c027 100644 --- a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/uri/WikibaseStyleStatementInlineUriHandler.java +++ b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/inline/uri/WikibaseStyleStatementInlineUriHandler.java @@ -4,7 +4,8 @@ import java.util.Locale; import java.util.UUID; -import org.apache.log4j.Logger; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; import com.bigdata.rdf.internal.InlineURIHandler; import com.bigdata.rdf.internal.impl.literal.AbstractLiteralIV; @@ -29,7 +30,7 @@ * @deprecated */ public class WikibaseStyleStatementInlineUriHandler extends InlineURIHandler { -private static final Logger log = Logger.getLogger(WikibaseStyleStatementInlineUriHandler.class); +private static final Logger log = LoggerFactory.getLogger(WikibaseStyleStatementInlineUriHandler.class); public WikibaseStyleStatementInlineUriHandler(String namespace) { super(namespace); @@ -99,7 +100,7 @@ i = i.shiftLeft(Long.SIZE).or(unsigned(u.getLeastSignificantBits())); return new XSDIntegerIV(i); } catch (IllegalArgumentException e) { - Logger.getLogger(WikibaseStyleStatementInlineUriHandler.class).warn("tmp", e); +log.warn("tmp", e); return null; } } diff --git a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/EmptyLabelServiceOptimizer.java b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/EmptyLabelServiceOptimizer.java index 44cb389..e5922bd 100644 --- a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/EmptyLabelServiceOptimizer.java +++ b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/label/EmptyLabelServiceOptimizer.java @@ -2,11 +2,12 @@ import java.util.List; -import org.apache.log4j.Logger; import org.openrdf.model.URI; import org.openrdf.model.impl.URIImpl; import org.openrdf.model.vocabulary.RDFS; import org.openrdf.model.vocabulary.SKOS; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; import org.wikidata.query.rdf.common.uri.Ontology; import org.wikidata.query.rdf.common.uri.SchemaDotOrg; @@ -36,7 +37,7 @@ */ @SuppressWarnings("rawtypes") public class EmptyLabelServiceOptimizer extends AbstractJoinGroupOptimizer { -private static final Logger log = Logger.getLogger(EmptyLabelServiceOptimizer.class); +