[jira] [Commented] (CASSANDRA-18144) org.apache.cassandra.db.compaction.CompactionStrategyManagerBoundaryReloadTest.testReload fails when running with TrieMemtables

2023-01-21 Thread Kamalesh Palanisamy (Jira)


[ 
https://issues.apache.org/jira/browse/CASSANDRA-18144?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17679501#comment-17679501
 ] 

Kamalesh Palanisamy commented on CASSANDRA-18144:
-

Submitted PR to fix this issue: https://github.com/apache/cassandra/pull/2107

> org.apache.cassandra.db.compaction.CompactionStrategyManagerBoundaryReloadTest.testReload
>  fails when running with TrieMemtables
> ---
>
> Key: CASSANDRA-18144
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18144
> Project: Cassandra
>  Issue Type: Bug
>  Components: CI
>Reporter: David Capwell
>Assignee: Kamalesh Palanisamy
>Priority: Normal
> Fix For: 4.x
>
>
> https://app.circleci.com/pipelines/github/dcapwell/cassandra/1771/workflows/1bd920c8-8568-44b3-9e8b-b152a73cf4fc/jobs/15393
> {code}
> java.lang.RuntimeException: Error setting schema for test (query was: alter 
> table cql_test_keyspace.table_01 with compaction = {'class': 
> 'SizeTieredCompactionStrategy', 'enabled': false})
>   at org.apache.cassandra.cql3.CQLTester.schemaChange(CQLTester.java:1222)
>   at org.apache.cassandra.cql3.CQLTester.alterTable(CQLTester.java:1009)
>   at 
> org.apache.cassandra.db.compaction.CompactionStrategyManagerBoundaryReloadTest.testReload(CompactionStrategyManagerBoundaryReloadTest.java:82)
> Caused by: java.lang.ClassCastException: 
> org.apache.cassandra.dht.ByteOrderedPartitioner$BytesToken cannot be cast to 
> org.apache.cassandra.dht.Murmur3Partitioner$LongToken
>   at 
> org.apache.cassandra.dht.Murmur3Partitioner$1.valueForToken(Murmur3Partitioner.java:68)
>   at 
> org.apache.cassandra.dht.Splitter$WeightedRange.totalTokens(Splitter.java:278)
>   at org.apache.cassandra.dht.Splitter.splitOwnedRanges(Splitter.java:129)
>   at 
> org.apache.cassandra.db.ColumnFamilyStore.localRangeSplits(ColumnFamilyStore.java:1504)
>   at 
> org.apache.cassandra.db.memtable.AbstractShardedMemtable.(AbstractShardedMemtable.java:65)
>   at 
> org.apache.cassandra.db.memtable.TrieMemtable.(TrieMemtable.java:142)
>   at 
> org.apache.cassandra.db.memtable.TrieMemtable$Factory.create(TrieMemtable.java:688)
>   at 
> org.apache.cassandra.db.ColumnFamilyStore.createMemtable(ColumnFamilyStore.java:1375)
>   at 
> org.apache.cassandra.db.ColumnFamilyStore$Flush.(ColumnFamilyStore.java:1173)
>   at 
> org.apache.cassandra.db.ColumnFamilyStore$Flush.(ColumnFamilyStore.java:1137)
>   at 
> org.apache.cassandra.db.ColumnFamilyStore.switchMemtable(ColumnFamilyStore.java:1000)
>   at 
> org.apache.cassandra.db.ColumnFamilyStore.switchMemtableIfCurrent(ColumnFamilyStore.java:981)
>   at 
> org.apache.cassandra.db.ColumnFamilyStore.switchMemtableOrNotify(ColumnFamilyStore.java:966)
>   at 
> org.apache.cassandra.db.ColumnFamilyStore.reload(ColumnFamilyStore.java:393)
> {code}
> First reported to slack: 
> https://the-asf.slack.com/archives/CK23JSY2K/p1673382016638189



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Updated] (CASSANDRA-18139) Revert changes to units output in FileUtils#stringifyFileSize

2023-01-21 Thread Ekaterina Dimitrova (Jira)


 [ 
https://issues.apache.org/jira/browse/CASSANDRA-18139?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ekaterina Dimitrova updated CASSANDRA-18139:

Summary: Revert changes to units output in FileUtils#stringifyFileSize  
(was: Revert changes to units output in FileUtils#stringifyFileSize post 
CASSANDRA-15234)

> Revert changes to units output in FileUtils#stringifyFileSize
> -
>
> Key: CASSANDRA-18139
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18139
> Project: Cassandra
>  Issue Type: Bug
>  Components: Local/Config
>Reporter: Ekaterina Dimitrova
>Assignee: Ekaterina Dimitrova
>Priority: Normal
> Fix For: 4.x
>
>
> As discussed in CASSANDRA-15234, FileUtils#stringifyFileSize is used in 
> nodetool output which can break people parsing the nodetool output



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522

2023-01-21 Thread Ekaterina Dimitrova (Jira)


[ 
https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17679489#comment-17679489
 ] 

Ekaterina Dimitrova commented on CASSANDRA-18181:
-

I just started the simulator tests in CircleCI as they are not added yet to 
Jenkins (CASSANDRA-18003) - they finished successfully 
[here|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2185/workflows/5370326f-a9b9-472a-a226-675b24201f3c]

> Fix tests post JDK-8210522
> --
>
> Key: CASSANDRA-18181
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18181
> Project: Cassandra
>  Issue Type: Bug
>  Components: CI
>Reporter: Ekaterina Dimitrova
>Assignee: Ekaterina Dimitrova
>Priority: Normal
> Fix For: 4.x
>
>
>  
> From JDK-8210522:
> {code:java}
> Core reflection has a filtering mechanism to hide security and integrity 
> sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). 
> The filtering mechanism has been used for several releases to hide security 
> sensitive fields such as System.security and Class.classLoader.
> This CSR proposes to extend the filters to hide fields from a number of 
> highly security sensitive classes in java.lang.reflect and java.lang.invoke.
> {code}
> We are using at a few places in our tests 
> {code:java}
> Field.class.getDeclaredField("modifiers");{code}
> This breaks as expected when tests are run with JDK17, example:
>  
> {code:java}
> java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers
>  at 
> org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79)
>  at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native 
> Method)
>  at 
> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
>  at 
> java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>  at java.base/java.lang.reflect.Method.invoke(Method.java:568)
>  at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>  at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>  at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>  at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24)
>  at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>  at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at 
> org.junit.runner.JUnitCore.run(JUnitCore.java:137)
>  at 
> com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69)
>  at 
> com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33)
>  at 
> com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221)
>  at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) 
> Caused by: java.lang.NoSuchFieldException: modifiers at 
> java.base/java.lang.Class.getDeclaredField(Class.java:2610) 
> at 
> org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70)
>  
> ... 15 more{code}
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Comment Edited] (CASSANDRA-18181) Fix tests post JDK-8210522

2023-01-21 Thread Ekaterina Dimitrova (Jira)


[ 
https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17679481#comment-17679481
 ] 

Ekaterina Dimitrova edited comment on CASSANDRA-18181 at 1/21/23 10:26 PM:
---

Wall of failures in DTests, probably it didn't download the updated DTest trunk 
branch

Just pushed it again 
[here|https://ci-cassandra.apache.org/view/patches/job/Cassandra-devbranch/2214/]


was (Author: e.dimitrova):
Wall of failures in DTests, probably it didn't download the updated DTest trunk 
branch

Just pushed it again 
[here|https://ci-cassandra.apache.org/view/patches/job/Cassandra-devbranch/2213/]

> Fix tests post JDK-8210522
> --
>
> Key: CASSANDRA-18181
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18181
> Project: Cassandra
>  Issue Type: Bug
>  Components: CI
>Reporter: Ekaterina Dimitrova
>Assignee: Ekaterina Dimitrova
>Priority: Normal
> Fix For: 4.x
>
>
>  
> From JDK-8210522:
> {code:java}
> Core reflection has a filtering mechanism to hide security and integrity 
> sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). 
> The filtering mechanism has been used for several releases to hide security 
> sensitive fields such as System.security and Class.classLoader.
> This CSR proposes to extend the filters to hide fields from a number of 
> highly security sensitive classes in java.lang.reflect and java.lang.invoke.
> {code}
> We are using at a few places in our tests 
> {code:java}
> Field.class.getDeclaredField("modifiers");{code}
> This breaks as expected when tests are run with JDK17, example:
>  
> {code:java}
> java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers
>  at 
> org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79)
>  at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native 
> Method)
>  at 
> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
>  at 
> java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>  at java.base/java.lang.reflect.Method.invoke(Method.java:568)
>  at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>  at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>  at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>  at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24)
>  at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>  at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at 
> org.junit.runner.JUnitCore.run(JUnitCore.java:137)
>  at 
> com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69)
>  at 
> com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33)
>  at 
> com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221)
>  at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) 
> Caused by: java.lang.NoSuchFieldException: modifiers at 
> java.base/java.lang.Class.getDeclaredField(Class.java:2610) 
> at 
> org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70)
>  
> ... 15 more{code}
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Comment Edited] (CASSANDRA-18179) Update current trunk build.xml and conf scripts for JDK17

2023-01-21 Thread Ekaterina Dimitrova (Jira)


[ 
https://issues.apache.org/jira/browse/CASSANDRA-18179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17679485#comment-17679485
 ] 

Ekaterina Dimitrova edited comment on CASSANDRA-18179 at 1/21/23 10:11 PM:
---

Overall looks good, I left just a few questions/comments on the commit. 
Further to testing J11 and J8 current CI, I can pull the change to my branch 
and test with JDK17 with the removed UDFs. I can get to this probably tomorrow
Considering trunk does not yet compile I guess we can just leave a comment 
about that in build.xml. And considering that I do not see it a breaking change 
that CCM is still not ready as we can't even compile...yet


was (Author: e.dimitrova):
Overall looks good, I left just a few questions/comments on the commit. 
Further to testing J11 and J8 current CI, I can pull the change to my branch 
and test with JDK17 with the removed UDFs.
Considering trunk does not yet compile I guess we can just leave a comment 
about that in build.xml. And considering that I do not see it a breaking change 
that CCM is still not ready as we can't even compile...yet

> Update current trunk build.xml and conf scripts for JDK17
> -
>
> Key: CASSANDRA-18179
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18179
> Project: Cassandra
>  Issue Type: Task
>  Components: Build
>Reporter: Ekaterina Dimitrova
>Assignee: Michael Semb Wever
>Priority: Normal
> Fix For: 4.x
>
>
> To support JDK17 we need to make changes to build.xml and our scripts. There 
> is a preliminary patch which makes a switch from JDK8+JDK11 to JDK11+JDK17 
> but it will need a change considering CASSANDRA-18133
> CC [~mck] 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Commented] (CASSANDRA-18179) Update current trunk build.xml and conf scripts for JDK17

2023-01-21 Thread Ekaterina Dimitrova (Jira)


[ 
https://issues.apache.org/jira/browse/CASSANDRA-18179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17679485#comment-17679485
 ] 

Ekaterina Dimitrova commented on CASSANDRA-18179:
-

Overall looks good, I left just a few questions/comments on the commit. 
Further to testing J11 and J8 current CI, I can pull the change to my branch 
and test with JDK17 with the removed UDFs.
Considering trunk does not yet compile I guess we can just leave a comment 
about that in build.xml. And considering that I do not see it a breaking change 
that CCM is still not ready as we can't even compile...yet

> Update current trunk build.xml and conf scripts for JDK17
> -
>
> Key: CASSANDRA-18179
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18179
> Project: Cassandra
>  Issue Type: Task
>  Components: Build
>Reporter: Ekaterina Dimitrova
>Assignee: Michael Semb Wever
>Priority: Normal
> Fix For: 4.x
>
>
> To support JDK17 we need to make changes to build.xml and our scripts. There 
> is a preliminary patch which makes a switch from JDK8+JDK11 to JDK11+JDK17 
> but it will need a change considering CASSANDRA-18133
> CC [~mck] 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Commented] (CASSANDRA-17964) Some tests are never executed due to naming violation - fix it and add checkstyle where applicable

2023-01-21 Thread Ekaterina Dimitrova (Jira)


[ 
https://issues.apache.org/jira/browse/CASSANDRA-17964?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17679484#comment-17679484
 ] 

Ekaterina Dimitrova commented on CASSANDRA-17964:
-

QQ: why should we @ignore the tests that are failing if they will have tickets 
anyway and we need to fix them? How many are those that fail? 

> Some tests are never executed due to naming violation - fix it and add 
> checkstyle where applicable
> --
>
> Key: CASSANDRA-17964
> URL: https://issues.apache.org/jira/browse/CASSANDRA-17964
> Project: Cassandra
>  Issue Type: Task
>  Components: Test/unit
>Reporter: Ruslan Fomkin
>Assignee: Stefan Miklosovic
>Priority: Normal
> Fix For: 3.0.x, 3.11.x, 4.0.x, 4.1.x, 4.x
>
>  Time Spent: 2h 50m
>  Remaining Estimate: 0h
>
> [BatchTests|https://github.com/apache/cassandra/blob/trunk/test/unit/org/apache/cassandra/cql3/BatchTests.java]
>  doesn't follow naming convention to be run as unit tests and, thus, is never 
> run.
> The rule in build expects names as `*Test`.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522

2023-01-21 Thread Ekaterina Dimitrova (Jira)


[ 
https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17679481#comment-17679481
 ] 

Ekaterina Dimitrova commented on CASSANDRA-18181:
-

Wall of failures in DTests, probably it didn't download the updated DTest trunk 
branch

Just pushed it again 
[here|https://ci-cassandra.apache.org/view/patches/job/Cassandra-devbranch/2213/]

> Fix tests post JDK-8210522
> --
>
> Key: CASSANDRA-18181
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18181
> Project: Cassandra
>  Issue Type: Bug
>  Components: CI
>Reporter: Ekaterina Dimitrova
>Assignee: Ekaterina Dimitrova
>Priority: Normal
> Fix For: 4.x
>
>
>  
> From JDK-8210522:
> {code:java}
> Core reflection has a filtering mechanism to hide security and integrity 
> sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). 
> The filtering mechanism has been used for several releases to hide security 
> sensitive fields such as System.security and Class.classLoader.
> This CSR proposes to extend the filters to hide fields from a number of 
> highly security sensitive classes in java.lang.reflect and java.lang.invoke.
> {code}
> We are using at a few places in our tests 
> {code:java}
> Field.class.getDeclaredField("modifiers");{code}
> This breaks as expected when tests are run with JDK17, example:
>  
> {code:java}
> java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers
>  at 
> org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79)
>  at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native 
> Method)
>  at 
> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
>  at 
> java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>  at java.base/java.lang.reflect.Method.invoke(Method.java:568)
>  at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>  at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>  at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>  at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24)
>  at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>  at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at 
> org.junit.runner.JUnitCore.run(JUnitCore.java:137)
>  at 
> com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69)
>  at 
> com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33)
>  at 
> com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221)
>  at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) 
> Caused by: java.lang.NoSuchFieldException: modifiers at 
> java.base/java.lang.Class.getDeclaredField(Class.java:2610) 
> at 
> org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70)
>  
> ... 15 more{code}
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Updated] (CASSANDRA-18179) Update current trunk build.xml and conf scripts for JDK17

2023-01-21 Thread Ekaterina Dimitrova (Jira)


 [ 
https://issues.apache.org/jira/browse/CASSANDRA-18179?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ekaterina Dimitrova updated CASSANDRA-18179:

Status: Review In Progress  (was: Patch Available)

> Update current trunk build.xml and conf scripts for JDK17
> -
>
> Key: CASSANDRA-18179
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18179
> Project: Cassandra
>  Issue Type: Task
>  Components: Build
>Reporter: Ekaterina Dimitrova
>Assignee: Michael Semb Wever
>Priority: Normal
> Fix For: 4.x
>
>
> To support JDK17 we need to make changes to build.xml and our scripts. There 
> is a preliminary patch which makes a switch from JDK8+JDK11 to JDK11+JDK17 
> but it will need a change considering CASSANDRA-18133
> CC [~mck] 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Updated] (CASSANDRA-18077) Add SpotBugs to the build

2023-01-21 Thread Ekaterina Dimitrova (Jira)


 [ 
https://issues.apache.org/jira/browse/CASSANDRA-18077?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ekaterina Dimitrova updated CASSANDRA-18077:

Reviewers:   (was: Ekaterina Dimitrova)

> Add SpotBugs to the build
> -
>
> Key: CASSANDRA-18077
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18077
> Project: Cassandra
>  Issue Type: Improvement
>  Components: Build, CI
>Reporter: David Capwell
>Priority: Normal
> Fix For: 4.x
>
> Attachments: spotbugs.html
>
>  Time Spent: 50m
>  Remaining Estimate: 0h
>
> When working on CASSANDRA-17178 I found that several classes were being 
> reported by the Simulator for not defining serializer version when they are 
> Serializable; this may cause issues for the Simulator so felt it would be 
> best to detect these earlier on before merging new ones.
> SpotBugs has a large set of checks, some more valuable than others for the 
> project; so we should maintain a curated list of issues to fail the build on, 
> and others to warn on.
> This topic was discussed in the following mail threads:
> * Should we add?: 
> https://lists.apache.org/thread/1ro1mvkpvt4vr24nw7dbpdlxo82mq3hz
> * Should we fix UTF-8 issues? 
> https://lists.apache.org/thread/sokxf46s7hyoxr9q4wm7dv3q2nm19nt3



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Updated] (CASSANDRA-16615) Incompatible JNA libraries cause unnecessary failures when using cassandra-all jar.

2023-01-21 Thread Michael Semb Wever (Jira)


 [ 
https://issues.apache.org/jira/browse/CASSANDRA-16615?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Semb Wever updated CASSANDRA-16615:
---
Reviewers:   (was: Michael Semb Wever)

> Incompatible JNA libraries cause unnecessary failures when using 
> cassandra-all jar.
> ---
>
> Key: CASSANDRA-16615
> URL: https://issues.apache.org/jira/browse/CASSANDRA-16615
> Project: Cassandra
>  Issue Type: Bug
>  Components: Local/Other
>Reporter: Doug Rohrer
>Priority: Normal
>
> There are some additional use-cases where lack of JNA libraries is causing 
> issues for users of cassandra-all libraries - cases where the native 
> libraries were not loaded but we should be able to continue should be handled 
> without throwing exceptions. Note that the Cassandra +server+ will not start 
> up without an appropriate/compatible JNA jar available, but users of the 
> cassandra-all libraries will have a more difficult time maintaining 
> compatibility across JNA versions as they deploy to, for example, Spark 
> setups that have newer/older JNA versions.
> Proposal:
> - Audit usages of `NativeLibrary`
> - Wherever possible, handle the -1 return values without failing +if 
> `DatabaseDescriptor.isClientInitialized()` returns true+.
> - Maintain existing failure behavior if the system is not in client mode but 
> the call returns -1.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Updated] (CASSANDRA-18077) Add SpotBugs to the build

2023-01-21 Thread Michael Semb Wever (Jira)


 [ 
https://issues.apache.org/jira/browse/CASSANDRA-18077?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Semb Wever updated CASSANDRA-18077:
---
Reviewers: Ekaterina Dimitrova  (was: Ekaterina Dimitrova, Michael Semb 
Wever)

> Add SpotBugs to the build
> -
>
> Key: CASSANDRA-18077
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18077
> Project: Cassandra
>  Issue Type: Improvement
>  Components: Build, CI
>Reporter: David Capwell
>Priority: Normal
> Fix For: 4.x
>
> Attachments: spotbugs.html
>
>  Time Spent: 50m
>  Remaining Estimate: 0h
>
> When working on CASSANDRA-17178 I found that several classes were being 
> reported by the Simulator for not defining serializer version when they are 
> Serializable; this may cause issues for the Simulator so felt it would be 
> best to detect these earlier on before merging new ones.
> SpotBugs has a large set of checks, some more valuable than others for the 
> project; so we should maintain a curated list of issues to fail the build on, 
> and others to warn on.
> This topic was discussed in the following mail threads:
> * Should we add?: 
> https://lists.apache.org/thread/1ro1mvkpvt4vr24nw7dbpdlxo82mq3hz
> * Should we fix UTF-8 issues? 
> https://lists.apache.org/thread/sokxf46s7hyoxr9q4wm7dv3q2nm19nt3



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Updated] (CASSANDRA-18179) Update current trunk build.xml and conf scripts for JDK17

2023-01-21 Thread Michael Semb Wever (Jira)


 [ 
https://issues.apache.org/jira/browse/CASSANDRA-18179?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Semb Wever updated CASSANDRA-18179:
---
Reviewers: Ekaterina Dimitrova

> Update current trunk build.xml and conf scripts for JDK17
> -
>
> Key: CASSANDRA-18179
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18179
> Project: Cassandra
>  Issue Type: Task
>  Components: Build
>Reporter: Ekaterina Dimitrova
>Assignee: Michael Semb Wever
>Priority: Normal
> Fix For: 4.x
>
>
> To support JDK17 we need to make changes to build.xml and our scripts. There 
> is a preliminary patch which makes a switch from JDK8+JDK11 to JDK11+JDK17 
> but it will need a change considering CASSANDRA-18133
> CC [~mck] 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Assigned] (CASSANDRA-18179) Update current trunk build.xml and conf scripts for JDK17

2023-01-21 Thread Michael Semb Wever (Jira)


 [ 
https://issues.apache.org/jira/browse/CASSANDRA-18179?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Semb Wever reassigned CASSANDRA-18179:
--

Assignee: Michael Semb Wever

> Update current trunk build.xml and conf scripts for JDK17
> -
>
> Key: CASSANDRA-18179
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18179
> Project: Cassandra
>  Issue Type: Task
>  Components: Build
>Reporter: Ekaterina Dimitrova
>Assignee: Michael Semb Wever
>Priority: Normal
> Fix For: 4.x
>
>
> To support JDK17 we need to make changes to build.xml and our scripts. There 
> is a preliminary patch which makes a switch from JDK8+JDK11 to JDK11+JDK17 
> but it will need a change considering CASSANDRA-18133
> CC [~mck] 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Updated] (CASSANDRA-18179) Update current trunk build.xml and conf scripts for JDK17

2023-01-21 Thread Michael Semb Wever (Jira)


 [ 
https://issues.apache.org/jira/browse/CASSANDRA-18179?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Semb Wever updated CASSANDRA-18179:
---
Test and Documentation Plan: CI
 Status: Patch Available  (was: Open)

> Update current trunk build.xml and conf scripts for JDK17
> -
>
> Key: CASSANDRA-18179
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18179
> Project: Cassandra
>  Issue Type: Task
>  Components: Build
>Reporter: Ekaterina Dimitrova
>Priority: Normal
> Fix For: 4.x
>
>
> To support JDK17 we need to make changes to build.xml and our scripts. There 
> is a preliminary patch which makes a switch from JDK8+JDK11 to JDK11+JDK17 
> but it will need a change considering CASSANDRA-18133
> CC [~mck] 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org



[jira] [Commented] (CASSANDRA-18179) Update current trunk build.xml and conf scripts for JDK17

2023-01-21 Thread Michael Semb Wever (Jira)


[ 
https://issues.apache.org/jira/browse/CASSANDRA-18179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17679443#comment-17679443
 ] 

Michael Semb Wever commented on CASSANDRA-18179:


Patch at 
https://github.com/apache/cassandra/compare/trunk...michaelsembwever:cassandra:mck/18179/trunk
 

This introduces jdk17, but keeps jdk8 (and keeps it as the default).
This allows us to incrementally progress on the jdk17 tickets.

> Update current trunk build.xml and conf scripts for JDK17
> -
>
> Key: CASSANDRA-18179
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18179
> Project: Cassandra
>  Issue Type: Task
>  Components: Build
>Reporter: Ekaterina Dimitrova
>Priority: Normal
> Fix For: 4.x
>
>
> To support JDK17 we need to make changes to build.xml and our scripts. There 
> is a preliminary patch which makes a switch from JDK8+JDK11 to JDK11+JDK17 
> but it will need a change considering CASSANDRA-18133
> CC [~mck] 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

-
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org