Also it costs us nothing to add it. On Wed, Jan 16, 2019 at 4:29 PM Jonathan Haddad <j...@jonhaddad.com> wrote:
> I'm +1 on the warning for two reasons. > > > A cqlsh warning only applies to those that create the sasi via cqlsh. > > 1. When people are creating their schemas in development, this is usually > the first step. You use the REPL to figure out what you need, then you > copy your schema somewhere else. The warning here should prevent a lot of > folks from making a serious mistake. > > 2. It's consistent with how we warn when people try to use materialized > views. > > > > > On Wed, Jan 16, 2019 at 2:15 PM Mick Semb Wever <m...@apache.org> wrote: > > > Regarding the warning, we might add it at least in 3.11, since for that > > version the property to enable SASI is going to be present but not > disabled > > by default. WDYT? > > > > > > I'm -0 on this. > > > > A single line warning in the logs on the sasi creation won't be noticed > by > > many users. > > A cqlsh warning only applies to those that create the sasi via cqlsh. > > And we're not talking about patching client drivers to generate a warning > > there. > > > > So I'd be happy with a yaml comment on the config flag explaining that > > it's a beta feature and that users should check open tickets and > understand > > current limitations on sasi before using them. > > > > regards, > > Mick > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@cassandra.apache.org > > For additional commands, e-mail: dev-h...@cassandra.apache.org > > > > > > -- > Jon Haddad > http://www.rustyrazorblade.com > twitter: rustyrazorblade >
