Skip to content

Conversation

@absurdfarce
Copy link
Contributor

@absurdfarce absurdfarce commented Aug 14, 2025

Aiming for a more modern version of #1901

<tinkerpop.version>3.5.6</tinkerpop.version>
<!-- when changing version also update version in LICENSE_binary -->
<slf4j.version>1.7.26</slf4j.version>
<slf4j.version>2.0.16</slf4j.version>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Have to bump slf4j version as well here in order to address compile errors re: unable to find org.slf4j.spi.LoggingEvent once we moved to logback 1.3.x. Per the discussion here it looks like slf4j 2.x is required for logback 1.3.x.

@absurdfarce
Copy link
Contributor Author

Bah, it's even worse than I originally thought. slf4j 2.x includes some new OSGi directives which appear to break our OSGi tests. I'm trying to find a fix via this project but haven't had any luck so far.

mavenBundle("org.ow2.asm", "asm-commons").version("9.6"),
mavenBundle("org.ow2.asm", "asm-util").version("9.6"),
mavenBundle("org.ow2.asm", "asm-tree").version("9.6"),
mavenBundle("org.ow2.asm", "asm-analysis").version("9.6"),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This code was discovered and implemented independently by @lukasz-antoniak . Huge hat tip to him for his contribution here!

@absurdfarce
Copy link
Contributor Author

Jenkins run confirms that failures in OSGi tests are resolved by the fix provided here by @lukasz-antoniak. At this point I'm calling this one good.

absurdfarce added a commit to datastax/dsbulk that referenced this pull request Dec 12, 2025
@lukasz-antoniak lukasz-antoniak self-assigned this Jan 13, 2026
@lukasz-antoniak lukasz-antoniak self-requested a review January 13, 2026 10:57
@lukasz-antoniak lukasz-antoniak removed their assignment Jan 13, 2026
patch by Bret McGuire; reviewed by Bret McGuire and Lukasz Antoniak
reference: apache#2051
@absurdfarce absurdfarce merged commit ad7a103 into apache:4.x Jan 13, 2026
@absurdfarce absurdfarce deleted the logback-updates branch January 13, 2026 21:22
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