[jira] [Commented] (SOLR-14484) NPE in ConcurrentUpdateHttp2SolrClient MDC logging
[ https://issues.apache.org/jira/browse/SOLR-14484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17112220#comment-17112220 ] ASF subversion and git services commented on SOLR-14484: Commit ec71a6b4540c0106d4bcb61e0d0d1e20c9f57973 in lucene-solr's branch refs/heads/branch_8x from David Smiley [ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=ec71a6b ] SOLR-14484: avoid putting null into MDC Co-authored-by: Andras Salamon (cherry picked from commit 2ac640f9d066ebd88f4b5ebd1036792bdbf171bc) > NPE in ConcurrentUpdateHttp2SolrClient MDC logging > -- > > Key: SOLR-14484 > URL: https://issues.apache.org/jira/browse/SOLR-14484 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) >Affects Versions: 8.4.1 >Reporter: Andras Salamon >Priority: Minor > Attachments: SOLR-14484-01.patch, SOLR-14484-02.patch > > > {{client.getBaseURL()}} can be null in {{ConcurrentUpdateHttp2SolrClient}} > which can cause problems in MDC logging. > We had the following error in the stacktrace. We were using Solr 8.4.1 from > lily hbase-indexer which still uses log4j 1.2: > {noformat} > Error from server at http://127.0.0.1:45895/solr/collection1: > java.lang.NullPointerException > at java.util.Hashtable.put(Hashtable.java:459) > at org.apache.log4j.MDC.put0(MDC.java:150) > at org.apache.log4j.MDC.put(MDC.java:85) > at org.slf4j.impl.Log4jMDCAdapter.put(Log4jMDCAdapter.java:67) > at org.slf4j.MDC.put(MDC.java:147) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.addRunner(ConcurrentUpdateHttp2SolrClient.java:346) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.waitForEmptyQueue(ConcurrentUpdateHttp2SolrClient.java:565) > {noformat} -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
[jira] [Commented] (SOLR-14484) NPE in ConcurrentUpdateHttp2SolrClient MDC logging
[ https://issues.apache.org/jira/browse/SOLR-14484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17112218#comment-17112218 ] ASF subversion and git services commented on SOLR-14484: Commit 2ac640f9d066ebd88f4b5ebd1036792bdbf171bc in lucene-solr's branch refs/heads/master from David Smiley [ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=2ac640f ] SOLR-14484: avoid putting null into MDC Co-authored-by: Andras Salamon > NPE in ConcurrentUpdateHttp2SolrClient MDC logging > -- > > Key: SOLR-14484 > URL: https://issues.apache.org/jira/browse/SOLR-14484 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) >Affects Versions: 8.4.1 >Reporter: Andras Salamon >Priority: Minor > Attachments: SOLR-14484-01.patch, SOLR-14484-02.patch > > > {{client.getBaseURL()}} can be null in {{ConcurrentUpdateHttp2SolrClient}} > which can cause problems in MDC logging. > We had the following error in the stacktrace. We were using Solr 8.4.1 from > lily hbase-indexer which still uses log4j 1.2: > {noformat} > Error from server at http://127.0.0.1:45895/solr/collection1: > java.lang.NullPointerException > at java.util.Hashtable.put(Hashtable.java:459) > at org.apache.log4j.MDC.put0(MDC.java:150) > at org.apache.log4j.MDC.put(MDC.java:85) > at org.slf4j.impl.Log4jMDCAdapter.put(Log4jMDCAdapter.java:67) > at org.slf4j.MDC.put(MDC.java:147) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.addRunner(ConcurrentUpdateHttp2SolrClient.java:346) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.waitForEmptyQueue(ConcurrentUpdateHttp2SolrClient.java:565) > {noformat} -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
[jira] [Commented] (SOLR-14484) NPE in ConcurrentUpdateHttp2SolrClient MDC logging
[ https://issues.apache.org/jira/browse/SOLR-14484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17112137#comment-17112137 ] Lucene/Solr QA commented on SOLR-14484: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || || || || || {color:brown} Prechecks {color} || | {color:red}-1{color} | {color:red} test4tests {color} | {color:red} 0m 0s{color} | {color:red} The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. {color} | || || || || {color:brown} master Compile Tests {color} || | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 50s{color} | {color:green} master passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 51s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 51s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} Release audit (RAT) {color} | {color:green} 0m 51s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} Check forbidden APIs {color} | {color:green} 0m 51s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} Validate source patterns {color} | {color:green} 0m 51s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 4m 43s{color} | {color:green} solrj in the patch passed. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 8m 28s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | JIRA Issue | SOLR-14484 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/13003470/SOLR-14484-02.patch | | Optional Tests | compile javac unit ratsources checkforbiddenapis validatesourcepatterns | | uname | Linux lucene1-us-west 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | ant | | Personality | /home/jenkins/jenkins-slave/workspace/PreCommit-SOLR-Build/sourcedir/dev-tools/test-patch/lucene-solr-yetus-personality.sh | | git revision | master / 57b7d8a8dbf | | ant | version: Apache Ant(TM) version 1.10.5 compiled on March 28 2019 | | Default Java | LTS | | Test Results | https://builds.apache.org/job/PreCommit-SOLR-Build/751/testReport/ | | modules | C: solr/solrj U: solr/solrj | | Console output | https://builds.apache.org/job/PreCommit-SOLR-Build/751/console | | Powered by | Apache Yetus 0.7.0 http://yetus.apache.org | This message was automatically generated. > NPE in ConcurrentUpdateHttp2SolrClient MDC logging > -- > > Key: SOLR-14484 > URL: https://issues.apache.org/jira/browse/SOLR-14484 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) >Affects Versions: 8.4.1 >Reporter: Andras Salamon >Priority: Minor > Attachments: SOLR-14484-01.patch, SOLR-14484-02.patch > > > {{client.getBaseURL()}} can be null in {{ConcurrentUpdateHttp2SolrClient}} > which can cause problems in MDC logging. > We had the following error in the stacktrace. We were using Solr 8.4.1 from > lily hbase-indexer which still uses log4j 1.2: > {noformat} > Error from server at http://127.0.0.1:45895/solr/collection1: > java.lang.NullPointerException > at java.util.Hashtable.put(Hashtable.java:459) > at org.apache.log4j.MDC.put0(MDC.java:150) > at org.apache.log4j.MDC.put(MDC.java:85) > at org.slf4j.impl.Log4jMDCAdapter.put(Log4jMDCAdapter.java:67) > at org.slf4j.MDC.put(MDC.java:147) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.addRunner(ConcurrentUpdateHttp2SolrClient.java:346) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.waitForEmptyQueue(ConcurrentUpdateHttp2SolrClient.java:565) > {noformat} -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
[jira] [Commented] (SOLR-14484) NPE in ConcurrentUpdateHttp2SolrClient MDC logging
[ https://issues.apache.org/jira/browse/SOLR-14484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17111826#comment-17111826 ] Andras Salamon commented on SOLR-14484: --- Yes, that's even simpler. Uploaded a new patch. > NPE in ConcurrentUpdateHttp2SolrClient MDC logging > -- > > Key: SOLR-14484 > URL: https://issues.apache.org/jira/browse/SOLR-14484 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) >Affects Versions: 8.4.1 >Reporter: Andras Salamon >Priority: Minor > Attachments: SOLR-14484-01.patch, SOLR-14484-02.patch > > > {{client.getBaseURL()}} can be null in {{ConcurrentUpdateHttp2SolrClient}} > which can cause problems in MDC logging. > We had the following error in the stacktrace. We were using Solr 8.4.1 from > lily hbase-indexer which still uses log4j 1.2: > {noformat} > Error from server at http://127.0.0.1:45895/solr/collection1: > java.lang.NullPointerException > at java.util.Hashtable.put(Hashtable.java:459) > at org.apache.log4j.MDC.put0(MDC.java:150) > at org.apache.log4j.MDC.put(MDC.java:85) > at org.slf4j.impl.Log4jMDCAdapter.put(Log4jMDCAdapter.java:67) > at org.slf4j.MDC.put(MDC.java:147) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.addRunner(ConcurrentUpdateHttp2SolrClient.java:346) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.waitForEmptyQueue(ConcurrentUpdateHttp2SolrClient.java:565) > {noformat} -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
[jira] [Commented] (SOLR-14484) NPE in ConcurrentUpdateHttp2SolrClient MDC logging
[ https://issues.apache.org/jira/browse/SOLR-14484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17111446#comment-17111446 ] David Smiley commented on SOLR-14484: - I propose simply using {{String.valueOf(client.getBaseURL())}} instead, and thus wind up with a simple "null" string in this case. WDYT? > NPE in ConcurrentUpdateHttp2SolrClient MDC logging > -- > > Key: SOLR-14484 > URL: https://issues.apache.org/jira/browse/SOLR-14484 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) >Affects Versions: 8.4.1 >Reporter: Andras Salamon >Priority: Minor > Attachments: SOLR-14484-01.patch > > > {{client.getBaseURL()}} can be null in {{ConcurrentUpdateHttp2SolrClient}} > which can cause problems in MDC logging. > We had the following error in the stacktrace. We were using Solr 8.4.1 from > lily hbase-indexer which still uses log4j 1.2: > {noformat} > Error from server at http://127.0.0.1:45895/solr/collection1: > java.lang.NullPointerException > at java.util.Hashtable.put(Hashtable.java:459) > at org.apache.log4j.MDC.put0(MDC.java:150) > at org.apache.log4j.MDC.put(MDC.java:85) > at org.slf4j.impl.Log4jMDCAdapter.put(Log4jMDCAdapter.java:67) > at org.slf4j.MDC.put(MDC.java:147) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.addRunner(ConcurrentUpdateHttp2SolrClient.java:346) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.waitForEmptyQueue(ConcurrentUpdateHttp2SolrClient.java:565) > {noformat} -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
[jira] [Commented] (SOLR-14484) NPE in ConcurrentUpdateHttp2SolrClient MDC logging
[ https://issues.apache.org/jira/browse/SOLR-14484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17107235#comment-17107235 ] Lucene/Solr QA commented on SOLR-14484: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || || || || || {color:brown} Prechecks {color} || | {color:red}-1{color} | {color:red} test4tests {color} | {color:red} 0m 0s{color} | {color:red} The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. {color} | || || || || {color:brown} master Compile Tests {color} || | {color:green}+1{color} | {color:green} compile {color} | {color:green} 1m 0s{color} | {color:green} master passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 52s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 52s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} Release audit (RAT) {color} | {color:green} 0m 52s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} Check forbidden APIs {color} | {color:green} 0m 52s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} Validate source patterns {color} | {color:green} 0m 52s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 4m 42s{color} | {color:green} solrj in the patch passed. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 8m 39s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | JIRA Issue | SOLR-14484 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/13002906/SOLR-14484-01.patch | | Optional Tests | compile javac unit ratsources checkforbiddenapis validatesourcepatterns | | uname | Linux lucene1-us-west 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | ant | | Personality | /home/jenkins/jenkins-slave/workspace/PreCommit-SOLR-Build/sourcedir/dev-tools/test-patch/lucene-solr-yetus-personality.sh | | git revision | master / 010168c57b3 | | ant | version: Apache Ant(TM) version 1.10.5 compiled on March 28 2019 | | Default Java | LTS | | Test Results | https://builds.apache.org/job/PreCommit-SOLR-Build/747/testReport/ | | modules | C: solr/solrj U: solr/solrj | | Console output | https://builds.apache.org/job/PreCommit-SOLR-Build/747/console | | Powered by | Apache Yetus 0.7.0 http://yetus.apache.org | This message was automatically generated. > NPE in ConcurrentUpdateHttp2SolrClient MDC logging > -- > > Key: SOLR-14484 > URL: https://issues.apache.org/jira/browse/SOLR-14484 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) >Affects Versions: 8.4.1 >Reporter: Andras Salamon >Priority: Minor > Attachments: SOLR-14484-01.patch > > > {{client.getBaseURL()}} can be null in {{ConcurrentUpdateHttp2SolrClient}} > which can cause problems in MDC logging. > We had the following error in the stacktrace. We were using Solr 8.4.1 from > lily hbase-indexer which still uses log4j 1.2: > {noformat} > Error from server at http://127.0.0.1:45895/solr/collection1: > java.lang.NullPointerException > at java.util.Hashtable.put(Hashtable.java:459) > at org.apache.log4j.MDC.put0(MDC.java:150) > at org.apache.log4j.MDC.put(MDC.java:85) > at org.slf4j.impl.Log4jMDCAdapter.put(Log4jMDCAdapter.java:67) > at org.slf4j.MDC.put(MDC.java:147) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.addRunner(ConcurrentUpdateHttp2SolrClient.java:346) > at > org.apache.solr.client.solrj.impl.ConcurrentUpdateHttp2SolrClient.waitForEmptyQueue(ConcurrentUpdateHttp2SolrClient.java:565) > {noformat} -- This message was sent by Atlassian Jira (v8.3.4#803005) - To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org