Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-7891

Don't write meaningless WSREP messages to stdout / error log

        Details

        • Type: Bug
        • Status: Closed
        • Priority: Minor
        • Resolution: Duplicate
        • Affects Version/s: 10.1.3
        • Fix Version/s: 10.1.4
        • Component/s: wsrep
        • Labels:
          None

          Description

          The script writes it every time when it gets started, even though it has no wsrep/galera configuration whatsoever. Please don't do it. 

          150331 15:45:27 mysqld_safe WSREP: Running position recovery with --log_error='/data/releases/mariadb-10.1.3-linux-x86_64/data/wsrep_recovery.dlREN7' --pid-file='/data/releases/mariadb-10.1.3-linux-x86_64/data/wheezy-64-recover.pid'
150331 15:45:30 mysqld_safe WSREP: Position recovery skipped

          It's useless noise for a standalone server.
          Please don't write "Running position recovery" if it is "skipped" anyway; and try not to write "skipped" either, unless it's skipped on a questionable reason.

            Gliffy Diagrams

              Attachments

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                serg Sergei Golubchik added a comment -

                I would generally prefer wsrep to be much less visible in 10.1+. Like, not add wsrep to the server version, etc. It made sense when we had a special "wsrep-patched" server version. But when wsrep is always in the server, it's kind of redundant to mark the server as "contains wsrep"

                Show
                serg Sergei Golubchik added a comment - I would generally prefer wsrep to be much less visible in 10.1+. Like, not add wsrep to the server version, etc. It made sense when we had a special "wsrep-patched" server version. But when wsrep is always in the server, it's kind of redundant to mark the server as "contains wsrep"
                Hide
                Permalink
                elenst Elena Stepanova added a comment -

                That's right, I also vote for removing it from the version.

                Show
                elenst Elena Stepanova added a comment - That's right, I also vote for removing it from the version.
                Hide
                Permalink
                nirbhay_c Nirbhay Choubey added a comment -

                Duplicate of MDEV-7067

                Show
                nirbhay_c Nirbhay Choubey added a comment - Duplicate of MDEV-7067

                  People

                  • Assignee:
                    nirbhay_c Nirbhay Choubey
                    Reporter:
                    elenst Elena Stepanova
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    3 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: