Given how low risk this is, I don't see an issue with backporting it and I'm sure the usefulness outweighs what risk there is. +1 (5.0.1 though, not 5.0.0)
Kind Regards, Brandon On Fri, Jul 26, 2024 at 11:52 AM Yifan Cai <yc25c...@gmail.com> wrote: > > Hi everyone, > > CASSANDRA-19800 is currently in the state of ready to be committed. Before > that, I want to propose backporting it to 4.0, 4.1 and 5.0. > > The ability to notify CQLSSTableWriter user when new sstables are produced is > especially useful for Cassandra Analytics and other consumers. The API is > more reliable than monitoring the file directory. > > That being said, I am aware that the patch is an improvement and trunk only. > I want to ask for an exemption on backporting the patch for two reasons. It > is useful for Cassandra Analytics. The patch is low risk for Cassandra server > as it only touches CQLSSTableWriter, which is only used by toolings. > > - Yifan