Re: [VOTE] Release Apache Cassandra 4.1.6

[J. D. Jordan]

I know of a few custom tracing implementations that implement a custom query handler to work.  No clue how maintained they still are. The elassandra project uses the custom query handler. DataStax uses the customer query handler for DSE (we maintain our own forks now, so can work around what ever happens). I would suggest we try to at the least keep compile time compatibility for the interface if possible.  If the original interface is just broken then we can of course vote to change it with a note added to NEWS.TXT about the breaking change. On Jul 30, 2024, at 10:54 AM, Jon Haddad <j...@jonhaddad.com> wrote:  This patch fixes a long standing issue that's the root cause of availability failures.  Even though folks can specify a custom query handler with the -D flag, the number of users impacted by this is going to be incredibly small.  On the other hand, the fix helps every single user of 4.1 that puts too much pressure on the cluster, which happens fairly regularly.   My POV is that it's a fairly weak argument that this is a public interface, but I don't consider it worth debating whether it is or not, because even if it is, this improves stability of the database for all users, so it's worth going in.  Let's not be dogmatic about fixes that help 99% of users because an incredibly small number that actually implement a custom query handler will need to make a trivial update in order to use the latest 4.1.6 dependency. Jon On Tue, Jul 30, 2024 at 8:09 AM J. D. Jordan <jeremiah.jor...@gmail.com> wrote: Given we allow a pluggable query handler implementation to be specified for the server with a -D during startup. So I would consider the query handler one of our public interfaces. On Jul 30, 2024, at 9:35 AM, Alex Petrov <al...@coffeenco.de> wrote:  Hi Tommy, Thank you for spotting this and bringing this to community's attention. I believe our primary interfaces are native and internode protocol, and CLI tools. Most interfaces are used to to abstract implementations internally. Few interfaces, such as DataType, Partitioner, and Triggers can be depended upon by external tools using Cassandra as a library. There is no official way to plug in a QueryHandler, so I did not consider it to be a part of our public API.  From [1]:  > These considerations are especially important for public APIs, including CQL, virtual tables, JMX, yaml, system properties, etc. Any planned additions must be carefully considered in the context of any existing APIs. Where possible the approach of any existing API should be followed.  Maybe we should have an exhaustive list of public APIs, and explicitly mention that native and internode protocols are included, alongside with nodetool command API and output, but also which classes/interfaces specifically should be evolved with care. Thank you, --Alex [1] https://cassandra.apache.org/_/development/index.html On Tue, Jul 30, 2024, at 10:56 AM, Tommy Stendahl via dev wrote: Hi, There is a change in the QueryHandler interface introduced by https://issues.apache.org/jira/browse/CASSANDRA-19534 Do we allow changes such changes between 4.1.5 and 4.1.6? CASSANDRA-19534 looks like a very good change so maybe there is an exception in this case? /Tommy -----Original Message----- From: Brandon Williams <brandonwilli...@apache.org> Reply-To: dev@cassandra.apache.org To: dev <dev@cassandra.apache.org> Subject: [VOTE] Release Apache Cassandra 4.1.6 Date: Mon, 29 Jul 2024 09:36:04 -0500 Proposing the test build of Cassandra 4.1.6 for release. sha1: b662744af59f3a3dfbfeb7314e29fecb93abfd80 Git: https://eur02.safelinks.protection.outlook.com/?url=""> Maven Artifacts: https://eur02.safelinks.protection.outlook.com/?url=""> The Source and Build Artifacts, and the Debian and RPM packages and repositories, are available here: https://eur02.safelinks.protection.outlook.com/?url=""> The vote will be open for 72 hours (longer if needed). Everyone who has tested the build is invited to vote. Votes by PMC members are considered binding. A vote passes if there are at least three binding +1s and no -1's. [1]: CHANGES.txt: https://eur02.safelinks.protection.outlook.com/?url=""> [2]: NEWS.txt: https://eur02.safelinks.protection.outlook.com/?url=""> Kind Regards, Brandon