Skip to content

MINOR: [CI][Java] Work around Maven stdout/stderr issue - #415

Merged
amoeba merged 2 commits into
apache:mainfrom
amoeba:ci/silence-slf4j-messages
Oct 15, 2026
Merged

amoeba merged 2 commits into
apache:mainfrom
amoeba:ci/silence-slf4j-messages

Conversation

@amoeba

@amoeba amoeba commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

Nightly Java Cookbook CI started failing in https://github.com/apache/arrow-cookbook/actions/runs/38504124957/job/115657570017 with extraneous log output,

Expected:
    column-1    column-2    column-3
    IRAQ    false    42
    JAPAN    false    43
    JORDAN    true    44
    KENYA    true    45
Got:
    SLF4J(W): No SLF4J providers were found.
    SLF4J(W): Defaulting to no-operation (NOP) logger implementation
    SLF4J(W): See https://www.slf4j.org/codes.html#noProviders for further details.
    SLF4J(W): Class path contains SLF4J bindings targeting slf4j-api versions 1.7.x or earlier.
    SLF4J(W): Ignoring binding found at [jar:file:/home/runner/.m2/repository/org/slf4j/slf4j-jdk14/1.7.30/slf4j-jdk14-1.7.30.jar!/org/slf4j/impl/StaticLoggerBinder.class]
    SLF4J(W): See https://www.slf4j.org/codes.html#ignoredBindings for an explanation.
    column-1    column-2    column-3
    IRAQ    false    42
    JAPAN    false    43
    JORDAN    true    44
    KENYA    true    45

It looks like a bug got shipped with Maven 3.10 that caused stderr to start showing up in stdout which caused test failures. See https://www.github.com/apache/maven/issues/13380.

This PR tries an agent-suggested approach which runs the doc example directly with java so stdout is clean.

@amoeba

amoeba commented Oct 14, 2026

Copy link
Copy Markdown
Member Author

@amoeba

amoeba commented Oct 14, 2026

Copy link
Copy Markdown
Member Author

The SLF4J thing was just a symptom of the real problem which is a change in JLine that Maven picked up: apache/maven#13380.

@amoeba
amoeba force-pushed the ci/silence-slf4j-messages branch from 446f744 to 4255bd8 Compare October 14, 2026 18:05
@amoeba amoeba changed the title MINOR: [CI][Java] Filter out SL4J messages MINOR: [CI][Java] Work around Maven stdout/stderr issue Oct 14, 2026
@amoeba
amoeba marked this pull request as ready for review October 15, 2026 20:52
@amoeba
amoeba requested a review from lidavidm October 15, 2026 20:53
@amoeba

amoeba commented Oct 15, 2026

Copy link
Copy Markdown
Member Author

This seems to work okay.

@amoeba
amoeba merged commit 6b0ef33 into apache:main Oct 15, 2026
9 checks passed
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