bbotella commented on PR #3562: URL: https://github.com/apache/cassandra/pull/3562#issuecomment-2598626924
Thanks everyone for the feedback and comments @jmckenzie-dev @Maxwell-Guo @smiklosovic ! I think I addressed all the comments that were left on the code. As per @jmckenzie-dev , let's go one by one. 1. That's interesting! Can you point me to other part of the code where we are doing such tests? I'd like to take a look at them as an example to have something that covers this scenario. 2. So, I don't think this feature will go out there in a release that won't come up with TCM as well right? Should we defer this scenario? 3. The CEP mentions interaction with currently preexisting guardrails. I don't think it proposes having the feature hidden behind a guardrail? 4. Json is covered with the integration tests in `testConstraintWithJsonInsert`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
