You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "jamesnetherton (via GitHub)" <gi...@apache.org> on 2026/01/21 14:29:01 UTC
[I] Remove cassandra-quarkus cq-maven-plugin BOM configuration for lz4-java [camel-quarkus]
jamesnetherton opened a new issue, #8188:
URL: https://github.com/apache/camel-quarkus/issues/8188
Since `lz4-java` is defined in `cassandra-quarkus-bom` it gets reported as a conflicting 'BOM overlap' since 1.3.1:
https://github.com/datastax/cassandra-quarkus/blob/main/bom/pom.xml#L63-L68
When the overlap warning is suppressed, `lz4-java` is then added to `camel-quarkus-bom` as `<optional>true</optional>`. Which is not desirable for us.
For now, I can fix these issues in CQ. Ideally we need `cassandra-quarkus` to upgrade to Quarkus 3.31 and remove `lz4-java` from `cassandra-quarkus-bom`. I had a quick attempt at doing this and it did not seem trivial at first glance.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Remove cassandra-quarkus cq-maven-plugin BOM configuration for lz4-java [camel-quarkus]
Posted by "dheeraj12347 (via GitHub)" <gi...@apache.org>.
dheeraj12347 commented on issue #8188:
URL: https://github.com/apache/camel-quarkus/issues/8188#issuecomment-3821922699
I checked the current main branch.
I can see at.yawk.lz4:lz4-java is excluded in the cq-maven-plugin configuration via resolutionExclude, with a TODO pointing to this issue.
I do not see any ignoredOverlap configuration for at.yawk.lz4:lz4-java anymore.
From my reading of the discussion, it looks like the ignoredOverlap part has already been removed. Is there any further change expected on the CQ side for this issue (for example, when upstream Cassandra / Quarkus are aligned), or can this be closed?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Remove cassandra-quarkus cq-maven-plugin BOM configuration for lz4-java [camel-quarkus]
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on issue #8188:
URL: https://github.com/apache/camel-quarkus/issues/8188#issuecomment-3789289588
> @ppalaga not sure if the build failure should be considered a bug in the `cq-maven-plugin` `bom-overlaps` feature?
>
> ```
> [ERROR] The following artifacts are managed in both org.apache.camel.quarkus:camel-quarkus-bom:3.31.0-SNAPSHOT and org.apache.camel.quarkus:camel-quarkus-bom:3.31.0-SNAPSHOT:
> [ERROR] - at.yawk.lz4:lz4-java 1.10.1 ✅ 1.10.1
> [ERROR]
> [ERROR] You may want to either remove the listed artifacts from org.apache.camel.quarkus:camel-quarkus-bom:3.31.0-SNAPSHOT or add them to <ignoredOverlaps>
> ```
>
> `at.yawk.lz4:lz4-java` is not defined directly in `camel-quarkus-bom`. It is now being pulled in via `quarkus-cassandra-bom`:
>
> https://github.com/datastax/cassandra-quarkus/blob/736a91524ce86496bf73992a3b21382fe59bb85f/bom/pom.xml#L63-L68
I would say `bom-overlaps` did the right thing not allowing `at.yawk.lz4:lz4-java` to end up in our flattened BOM.
> For now, I can fix these issues in CQ.
I can see you did two things to workaround the problem on our side:
1. `<resolutionExclude>at.yawk.lz4:lz4-java:*:*:*:*</resolutionExclude>`
and
2. `<ignoredOverlap>at.yawk.lz4:lz4-java</ignoredOverlap>`
Ideally we need `cassandra-quarkus` to upgrade to Quarkus 3.31 and remove `lz4-java` from `cassandra-quarkus-bom`. I had a quick attempt at doing this and it did not seem trivial at first glance.
Out of those, only the first one really does what we want: not to take over the `at.yawk.lz4:lz4-java` entry from Cassandra BOM to our flattened and minimized BOM.
`<ignoredOverlap>at.yawk.lz4:lz4-java</ignoredOverlap>` can be removed.
> Ideally we need `cassandra-quarkus` to upgrade to Quarkus 3.31 and remove `lz4-java` from `cassandra-quarkus-bom`. I had a quick attempt at doing this and it did not seem trivial at first glance.
Yeah, fixing it in `cassandra-quarkus-bom` would be ideal.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Remove cassandra-quarkus cq-maven-plugin BOM configuration for lz4-java [camel-quarkus]
Posted by "dheeraj12347 (via GitHub)" <gi...@apache.org>.
dheeraj12347 commented on issue #8188:
URL: https://github.com/apache/camel-quarkus/issues/8188#issuecomment-3821870490
I’d like to work on this. I’ll remove the unnecessary ignoredOverlap configuration for at.yawk.lz4:lz4-java and verify the BOM build.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Remove cassandra-quarkus cq-maven-plugin BOM configuration for lz4-java [camel-quarkus]
Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton closed issue #8188: Remove cassandra-quarkus cq-maven-plugin BOM configuration for lz4-java
URL: https://github.com/apache/camel-quarkus/issues/8188
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Remove cassandra-quarkus cq-maven-plugin BOM configuration for lz4-java [camel-quarkus]
Posted by "ppalaga (via GitHub)" <gi...@apache.org>.
ppalaga commented on issue #8188:
URL: https://github.com/apache/camel-quarkus/issues/8188#issuecomment-3822195112
> I do not see any ignoredOverlap configuration for at.yawk.lz4:lz4-java anymore.
Yeah, that's right, as @jamesnetherton has removed it in between, because it was not needed to workaround the overlap issue. It was enough to add `<resolutionExclude>at.yawk.lz4:lz4-java:*:*:*:*</resolutionExclude>` to wokaround it. So `<resolutionExclude>at.yawk.lz4:lz4-java:*:*:*:*</resolutionExclude>` should be removed now that we are on a Quarkus version managing `at.yawk.lz4:lz4-java`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org