adelapena commented on pull request #1321: URL: https://github.com/apache/cassandra/pull/1321#issuecomment-984954385
Thanks for the review. I think I have addressed most of the comments, particularly the ones about dropping those features in the framework that are not used in by the included guardrails. The included guardrails (number of tables, indexes, etc.) were some simple guardrails taken from the CEP to show how the framework is used, so we could place this prototype into a feature branch, and then continue adding more guardrails using the framework, and probably do some final cleanup before merging into trunk. It seems that instead this has gone in the direction of adding a few guardrails and the (mostly) minimum part of the framework to make them work, so I'm wondering if we should commit this prototype directly to trunk when we are ready, given that it gives us some new features and it doesn't add unneeded code, wdyt? -- 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]
