Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MDEV-33763 : Buffered error logging for Galera #472

Open
wants to merge 1 commit into
base: 11.7
Choose a base branch
from

Conversation

janlindstrom
Copy link

This is adaptation of feature by Zsolt Parragi [email protected] for Percona XtraDB Cluster to MariaDB. Original feature is documented at https://www.percona.com/blog/introducing-buffered-error-logging-in-percona-server-for-mysql/.

In this adaptation variable names are changed and integration to server code is different.

Commit adds new variables
wsrep_buffered_error_log_buffer_size ulong default 0 dynamic
Size of buffer used in buffered error logging. Because
wsrep debug setting produces large number of messages
it is recomended to set this variable around 1-100Mb.

wsrep_buffered_error_log_filename string default NULL static
  Name of the file where buffered error logging is done.

wsrep_buffered_error_log_file_size ulong default 0 dynamic
  Maximum size of one buffered error log file. This variable
  should be configured to be 10-100 times bigger than buffer
  size.

wsrep_buffered_error_log_rotations uint default 9 dynamic.
  Number of buffered error log files kept before deleting
  them.

When wsrep_buffered_error_log_buffer_size is specified and non zero, a buffer is
allocated with the specified size, and log messages are
written there. This includes also wsrep debug messages.

When this buffer gets full, the server shuts down, or crashes, the
buffer is written to wsrep_buffered_error_log_filename.

When log files reaches configured size limit log is rotated and
only configured amount of log files are kept.

This is useful in situations where debug logs create lots of output and
slow down normal server operation, but are helpful for figuring out
issues, as writing to a memory buffer has less overhead.

Warning: When this feature is used server will write big number
of messages to error log. Therefore, it is recomended to store
these logs on separate location to avoid filling up your
database disk. Furthermore, it is recomended to set up
logrotate to delete old log files.

Detailed changes:

wsrep.h
Log printing is now controlled by wsrep_debug_mode variable and
controlled by wsrep_debug and wsrep_buffered_error_log_buffer_size
variables.

sql/log.cc
Added sql_print_debug to log wsrep debug messages (WSREP_DEBUG()).

print_buffer_to_file
Actual buffered error logging if enabled will happen in this
function.

sql/mysqld.cc
Buffered error logging is initialized and shutdown here if it is
configured.

sql/signal_handler.cc
If buffered error logging is enabled remaining buffer is written
before continuing signal handling.

sql/sys_vars.cc
New variables are defined here.

sql/wsrep_buffered_error_log.cc
sql/wsrep_buffered_error_log.h
Buffered error log implementation using logger API (mysys/file_logger.cc).

sql/wsrep_var.cc
New variables update and check functions.

mysys/file_logger.cc
Added support for buffered error logging, file name changing and
buffer resizing.

@janlindstrom janlindstrom self-assigned this Jan 8, 2025
@@ -0,0 +1,38 @@
# ==== Purpose ====
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this file used somewhere?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its seems it is not used

@@ -0,0 +1,105 @@
/* Copyright (C) 2012 Monty Program Ab
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In plugin/server_audit we have pretty much duplicated service_logger.h and file_logger.c?
Is that correct?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They are not same, I did not want to change audit log code, so I made copy of original API and used new API on my implementation that uses C++. Audit plugin uses C and changing that is out of scope for this PR.

Copy link

@sciascid sciascid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two of the new tests are not working:
galera.galera_var_buffered_error_log_file_size
galera_var_buffered_error_log_buffer_size

And this one needs to be re-recorded:
galera.galera_defaults

ON_CHECK(wsrep_buffered_error_log_filename_check),
ON_UPDATE(wsrep_buffered_error_log_filename_update));

static Sys_var_ulonglong Sys_wsrep_buffered_error_log_buffer_size(
Copy link

@sciascid sciascid Feb 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why would one want to set the buffer size? And what would be a good number?
Can't we just pick a fixed number, based on some basic performance testing?`

Copy link
Author

@janlindstrom janlindstrom Feb 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Naturally we could, buffer size configuration depends wsrep-debug level, higher lever produces more messages. In wsrep-debug it is around 2000kb/s (sysbench). I use this variable to indicate that buffered error logging is used, naturally we could use filename as well.

@janlindstrom janlindstrom force-pushed the 11.7-wsrep-buffered-error-logging branch from c374398 to 9101836 Compare February 7, 2025 07:39
This is adaptation of feature by Zsolt Parragi <[email protected]>
for Percona XtraDB Cluster to MariaDB. Original feature is documented at
https://www.percona.com/blog/introducing-buffered-error-logging-in-percona-server-for-mysql/.

In this adaptation variable names are changed and integration to server
code is different.

Commit adds new variables
    wsrep_buffered_error_log_buffer_size ulong default 0 dynamic
      Size of buffer used in buffered error logging. Because
      wsrep debug setting produces large number of messages
      it is recomended to set this variable around 1-100Mb.

    wsrep_buffered_error_log_filename string default NULL static
      Name of the file where buffered error logging is done.

    wsrep_buffered_error_log_file_size ulong default 0 dynamic
      Maximum size of one buffered error log file. This variable
      should be configured to be 10-100 times bigger than buffer
      size.

    wsrep_buffered_error_log_rotations uint default 9 dynamic.
      Number of buffered error log files kept before deleting
      them.

 When wsrep_buffered_error_log_buffer_size is specified and non zero, a buffer is
 allocated with the specified size, and log messages are
 written there. This includes also wsrep debug messages.

 When this buffer gets full, the server shuts down, or crashes, the
 buffer is written to wsrep_buffered_error_log_filename.

 When log files reaches configured size limit log is rotated and
 only configured amount of log files are kept.

 This is useful in situations where debug logs create lots of output and
 slow down normal server operation, but are helpful for figuring out
 issues, as writing to a memory buffer has less overhead.

 Warning: When this feature is used server will write big number
 of messages to error log. Therefore, it is recomended to store
 these logs on separate location to avoid filling up your
 database disk. Furthermore, it is recomended to set up
 logrotate to delete old log files.

Detailed changes:

wsrep.h
  Log printing is now controlled by wsrep_debug_mode variable and
  controlled by wsrep_debug and wsrep_buffered_error_log_buffer_size
  variables.

sql/log.cc
  Added sql_print_debug to log wsrep debug messages (WSREP_DEBUG()).

print_buffer_to_file
  Actual buffered error logging if enabled will happen in this
  function.

sql/mysqld.cc
  Buffered error logging is initialized and shutdown here if it is
  configured.

sql/signal_handler.cc
  If buffered error logging is enabled remaining buffer is written
  before continuing signal handling.

sql/sys_vars.cc
  New variables are defined here.

sql/wsrep_buffered_error_log.cc
sql/wsrep_buffered_error_log.h
  Buffered error log implementation using logger API (mysys/file_logger.cc).

sql/wsrep_var.cc
  New variables update and check functions.

mysys/file_logger.cc
  Added support for buffered error logging, file name changing and
  buffer resizing.
@janlindstrom janlindstrom force-pushed the 11.7-wsrep-buffered-error-logging branch from 9101836 to 8bee913 Compare February 7, 2025 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants