Solr
  1. Solr
  2. SOLR-4807

zkcli - log4j is not configured, so no logging happens

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 4.3
    • Fix Version/s: 4.3.1, 4.4, 6.0
    • Component/s: SolrCloud
    • Labels:
      None

      Description

      When executing the zkcli script from the 4.3.0 release, a log4j error message is displayed about no appenders, because there is no log4j configuration. As a result of this problem, none of the normal zkcli stuff gets logged.

      A minimal config needs to be present so that it behaves like it did before the logging change.

      1. SOLR-4807.patch
        2 kB
        Shawn Heisey
      2. SOLR-4807.patch
        2 kB
        Shawn Heisey
      3. SOLR-4807.patch
        2 kB
        Shawn Heisey

        Activity

        Hide
        Shawn Heisey added a comment -

        "Fix Version/s" includes 4.3.1, as this will probably be a small and safe fix.

        Show
        Shawn Heisey added a comment - "Fix Version/s" includes 4.3.1, as this will probably be a small and safe fix.
        Hide
        Shawn Heisey added a comment -

        I've assigned the issue to myself, but anyone who wants it is free to take it.

        Show
        Shawn Heisey added a comment - I've assigned the issue to myself, but anyone who wants it is free to take it.
        Hide
        Shawn Heisey added a comment -

        Patch against trunk that sends logging from zkCLI at INFO to stderr, which is what it did before the log4j change.

        This also fixes a bug that prevented zkcli.bat from working on windows.

        Show
        Shawn Heisey added a comment - Patch against trunk that sends logging from zkCLI at INFO to stderr, which is what it did before the log4j change. This also fixes a bug that prevented zkcli.bat from working on windows.
        Hide
        Shalin Shekhar Mangar added a comment -

        +1

        Show
        Shalin Shekhar Mangar added a comment - +1
        Hide
        Shawn Heisey added a comment -

        New patch with cosmetic formatting updates, a CHANGES.txt entry in the 4.3.1 section, and svn eol-style on the new properties file.

        I would appreciate a quick review and a note from Shalin Shekhar Mangar about whether it is OK to include in 4.3.1.

        Show
        Shawn Heisey added a comment - New patch with cosmetic formatting updates, a CHANGES.txt entry in the 4.3.1 section, and svn eol-style on the new properties file. I would appreciate a quick review and a note from Shalin Shekhar Mangar about whether it is OK to include in 4.3.1.
        Hide
        Shalin Shekhar Mangar added a comment -

        I should have clarified. My +1 was for including it in 4.3.1

        Having no logs is annoying but zkcli not working on windows is more serious and should also be mentioned in the change log.

        Show
        Shalin Shekhar Mangar added a comment - I should have clarified. My +1 was for including it in 4.3.1 Having no logs is annoying but zkcli not working on windows is more serious and should also be mentioned in the change log.
        Hide
        Shawn Heisey added a comment -

        Updated CHANGES.txt to include note about the fix for Windows. This is the version that I am committing and backporting.

        Show
        Shawn Heisey added a comment - Updated CHANGES.txt to include note about the fix for Windows. This is the version that I am committing and backporting.
        Hide
        Commit Tag Bot added a comment -

        [trunk commit] elyograg
        http://svn.apache.org/viewvc?view=revision&revision=1481220

        SOLR-4807: zkcli fix for log4j and Windows

        Show
        Commit Tag Bot added a comment - [trunk commit] elyograg http://svn.apache.org/viewvc?view=revision&revision=1481220 SOLR-4807 : zkcli fix for log4j and Windows
        Hide
        Commit Tag Bot added a comment -

        [branch_4x commit] elyograg
        http://svn.apache.org/viewvc?view=revision&revision=1481222

        SOLR-4807: zkcli fix for log4j and Windows (merge trunk r1481220)

        Show
        Commit Tag Bot added a comment - [branch_4x commit] elyograg http://svn.apache.org/viewvc?view=revision&revision=1481222 SOLR-4807 : zkcli fix for log4j and Windows (merge trunk r1481220)
        Hide
        Commit Tag Bot added a comment -

        [lucene_solr_4_3 commit] elyograg
        http://svn.apache.org/viewvc?view=revision&revision=1481224

        SOLR-4807: zkcli fix for log4j and Windows (merge trunk r1481220)

        Show
        Commit Tag Bot added a comment - [lucene_solr_4_3 commit] elyograg http://svn.apache.org/viewvc?view=revision&revision=1481224 SOLR-4807 : zkcli fix for log4j and Windows (merge trunk r1481220)
        Hide
        Shalin Shekhar Mangar added a comment -

        Bulk close after 4.3.1 release

        Show
        Shalin Shekhar Mangar added a comment - Bulk close after 4.3.1 release

          People

          • Assignee:
            Shawn Heisey
            Reporter:
            Shawn Heisey
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development