Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-3541

Wrong placeholder '{}' in logs.

VotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Minor
    • Resolution: Duplicate
    • 3.5.5
    • None
    • server

    Description

      In the method 'org.apache.zookeeper.client.ZooKeeperSaslClient#respondToServer', 

       LOG.error("SASL authentication failed using login context '"
 + this.getLoginContext()
 + "' with exception: {}", e); 

      // I think '{}' above is wrong. It's redundant.

       

      Attachments

        Issue Links

        is superceded by

        Improvement - An improvement or enhancement to an existing feature or task. ZOOKEEPER-3509 Revisit log format

        • Major - Major loss of function.
        • Resolved
        links to

        Web Link GitHub Pull Request #1084

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            xiaodong.hu hu xiaodong
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 1h 50m
                1h 50m

                Slack

                  Issue deployment