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.

Sign up for GitHub

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

Jump to bottom

Move async support to log4j-async-logger #2241

Merged
merged 8 commits into from
Jan 29, 2024
Merged

Move async support to log4j-async-logger #2241

merged 8 commits into from
Jan 29, 2024

Commits on Jan 29, 2024

  1. Inline EventRoute methods

    @ppkarwasz
    ppkarwasz committed Jan 29, 2024
    Configuration menu
    Copy the full SHA
    44613d0 View commit details
    Browse the repository at this point in the history

  2. Move async logger tests to log4j-async-logger 

    We move all the tests using `AsyncLogger` and `AsyncLoggerConfig` from
`log4j-core-test` to a new artifact `log4j-async-logger` in preparation
for splitting async support out of `log4j-core`.
    @ppkarwasz
    ppkarwasz committed Jan 29, 2024
    Configuration menu
    Copy the full SHA
    f5af863 View commit details
    Browse the repository at this point in the history

  3. Remove instance of checks for async components 

    This PR removes all occurences of `instanceof Async*` that might prevent
the separation of the async support.

The removal requires a couple of new API methods and binary incompatible
changes.
    @ppkarwasz
    ppkarwasz committed Jan 29, 2024
    Configuration menu
    Copy the full SHA
    4e732f1 View commit details
    Browse the repository at this point in the history

  4. Last tweaks to tests before moving async support 

    A couple more tests require changes to be independent from async loggers
and logger configurations.

The `CoreContextSelectors` class is not used, so we remove it.
    @ppkarwasz
    ppkarwasz committed Jan 29, 2024
    Configuration menu
    Copy the full SHA
    b69110b View commit details
    Browse the repository at this point in the history

  5. Move async support to log4j-async-logger 

    This moves the main code for `AsyncLogger` and `AsyncLoggerConfig` to
`log4j-async-logger`.

Further tasks such as proper encapsulation into internal packages and
removal of the optional dependency guards, will be performed in a future
task.
    @ppkarwasz
    ppkarwasz committed Jan 29, 2024
    Configuration menu
    Copy the full SHA
    a3cf42d View commit details
    Browse the repository at this point in the history

  6. Fix failing tests

    @ppkarwasz
    ppkarwasz committed Jan 29, 2024
    Configuration menu
    Copy the full SHA
    1fc3913 View commit details
    Browse the repository at this point in the history

  7. Add changelog for splitting off log4j-async-logger

    @ppkarwasz
    ppkarwasz committed Jan 29, 2024
    Configuration menu
    Copy the full SHA
    fd526f1 View commit details
    Browse the repository at this point in the history

  8. Remove references to async context selector in log4j-script

    @ppkarwasz
    ppkarwasz committed Jan 29, 2024
    Configuration menu
    Copy the full SHA
    4ad805a View commit details
    Browse the repository at this point in the history