Re: [DISCUSS] CEP-23: Enhancement for Sparse Data Serialization

2022-09-07 Thread Claude Warren via dev
I have looked through the code mentioned.  What I found in the ColumnSerializer was the use of VInt encoding.  Are you proposing switching directly to VInt encoding for sizes rather than one of the other encodings?  Using a -2 as the first length to signal that the new encoding is in use so tha

Re: [DISCUSS] CEP-20: Dynamic Data Masking

2022-09-07 Thread Jeremiah D Jordan
A > On Sep 7, 2022, at 8:58 AM, Benedict wrote: > > Well, I am not convinced these changes will materially impact the outcome, > but at least we’ll have some extra fun collating the votes. > > >> On 7 Sep 2022, at 14:05, Andrés de la Peña wrote: >> >>  >> The poll makes sense to me. I woul

Re: [DISCUSS] CEP-20: Dynamic Data Masking

2022-09-07 Thread Benedict
Well, I am not convinced these changes will materially impact the outcome, but at least we’ll have some extra fun collating the votes. > On 7 Sep 2022, at 14:05, Andrés de la Peña wrote: > >  > The poll makes sense to me. I would slightly change it to: > > A) We shouldn't prefer neither appr

Re: [DISCUSS] CEP-20: Dynamic Data Masking

2022-09-07 Thread Berenguer Blasi
A. I agree the implementor's preference is an important aspect to take into account. On 7/9/22 15:23, Ekaterina Dimitrova wrote: A On Wed, 7 Sep 2022 at 9:05, Andrés de la Peña wrote: The poll makes sense to me. I would slightly change it to: A) We shouldn't prefer neither approac

Re: [DISCUSS] CEP-20: Dynamic Data Masking

2022-09-07 Thread Ekaterina Dimitrova
A On Wed, 7 Sep 2022 at 9:05, Andrés de la Peña wrote: > The poll makes sense to me. I would slightly change it to: > > A) We shouldn't prefer neither approach, and I agree to the implementor > selecting the table schema approach for this CEP > B) We should prefer the view approach, but I am not

Re: [DISCUSS] CEP-20: Dynamic Data Masking

2022-09-07 Thread Andrés de la Peña
The poll makes sense to me. I would slightly change it to: A) We shouldn't prefer neither approach, and I agree to the implementor selecting the table schema approach for this CEP B) We should prefer the view approach, but I am not opposed to the implementor selecting the table schema approach for

Re: [DISCUSS] CEP-20: Dynamic Data Masking

2022-09-07 Thread Claude Warren via dev
My vote is B On 07/09/2022 13:12, Benedict wrote: I’m not convinced there’s been adequate resolution over which approach is adopted. I know you have expressed a preference for the table schema approach, but the weight of other opinion so far appears to be against this approach - even if it is

Re: [DISCUSS] CEP-20: Dynamic Data Masking

2022-09-07 Thread Benedict
I’m not convinced there’s been adequate resolution over which approach is adopted. I know you have expressed a preference for the table schema approach, but the weight of other opinion so far appears to be against this approach - even if it is broadly adopted by other databases. I will note that

Re: [DISCUSS] CEP-20: Dynamic Data Masking

2022-09-07 Thread Andrés de la Peña
If nobody has more concerns regarding the CEP I will start the vote tomorrow. On Wed, 31 Aug 2022 at 13:18, Andrés de la Peña wrote: > Is there enough support here for VIEWS to be the implementation strategy >> for displaying masking functions? > > > I'm not sure that views should be "the" strat

Re: [Marketing] For Review: Learn How CommitLog Works in Apache Cassandra

2022-09-07 Thread Chris Thornett
Amends have been made, and we've closed the review period. Thanks to everyone for your help on this one! On Wed, Aug 31, 2022 at 3:16 PM Chris Thornett wrote: > Update: Some sections have been flagged by the PMC for further revision. > Author, Alex Sorokoumov will review on 2 September. > > Revi