[
https://issues.apache.org/jira/browse/CASSANDRA-18700?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Brandon Williams updated CASSANDRA-18700:
-----------------------------------------
Reviewers: Brandon Williams
> Remove AssertJ annotations from the production code
> ---------------------------------------------------
>
> Key: CASSANDRA-18700
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18700
> Project: Cassandra
> Issue Type: Bug
> Components: Build
> Reporter: Stefan Miklosovic
> Assignee: Stefan Miklosovic
> Priority: Normal
> Fix For: 4.0.x, 4.1.x, 5.x
>
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> We are using VisibleForTesting annotation from AssertJ in the production
> code. At the same time, we are not shipping assertj library in the tarball.
> The way it works is that we depend on assertj with "provided" scope and then
> we will not ship the lib. It is possible, because the default RetentionPolicy
> for that annotation is CLASS which means that while we compile with the
> annotation, it does not need to be there in runtime ...
> There is VisibleForTesting annotation from Guava which is used extensively.
> Usage of VisibleForTesting from AssertJ was used by mistake as that probably
> popped up as the first import to add when doing code completion in IDEA.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
