Re: git: 906748d208d3 - main - newsyslog(8): Add option to globally override compression method.

In reply to: Xin LI : "git: 906748d208d3 - main - newsyslog(8): Add option to globally override compression method."
Go to: [ bottom of page ] [ top of archives ] [ this month ]

From: Enji Cooper <yaneurabeya_at_gmail.com>
Date: Sat, 23 Dec 2023 15:14:29 UTC

> On Dec 22, 2023, at 22:53, Xin LI <delphij@freebsd.org> wrote:
> 
> The branch main has been updated by delphij:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=906748d208d356a59538dc009a8ad5db040968d9
> 
> commit 906748d208d356a59538dc009a8ad5db040968d9
> Author:     Xin LI <delphij@FreeBSD.org>
> AuthorDate: 2023-12-23 06:46:33 +0000
> Commit:     Xin LI <delphij@FreeBSD.org>
> CommitDate: 2023-12-23 06:46:33 +0000
> 
>    newsyslog(8): Add option to globally override compression method.
> 
>    Historically, newsyslog compressed rotated log files to save disk space.
>    This was useful in the early days. However, with modern file systems like
>    ZFS offering native compression, and with the availability of larger hard
>    drives, the benefits of additional compression have become less significant.
>    This is particularly true considering the inconvenience of decompressing
>    log files when searching for specific patterns.
> 
>    Additionally, the original implementation of compression methods was not
>    future-proof.  As a result, we have redefined the J, X, Y, Z flags to
>    signify "treat the file as compressible" rather than "compress the file
>    with that specific method."
> 
>    A new command-line option, -c, has been introduced to allow overriding
>    these settings in a more future-proof way. The available choices are:
> 
>     * none                  - do not compress, regardless of flag.
>     * legacy                - historical behavior: J=bzip2, X=xz, Y=zstd, Z=gzip.
>     * bzip2, xz, zstd, gzip - apply the specified compression method.
> 
>    Currently, the default is set to 'legacy' to preserve historical behavior.
>    However, our intention is to change this default to 'none' in FreeBSD 15.0.
> 
>    Additionally, this update changes the default settings for zstd to use
>    multithreading and long-range options, better aligning with its intended
>    use.
> 
>    Inspired by D42961 .
> 
>    Reviewed by:    debdrup (earlier version, mdoc(7))
>    MFC after:      1 week
>    Differential Revision: https://reviews.freebsd.org/D43165

Relnotes: yes ?