michaelsembwever commented on PR #2282: URL: https://github.com/apache/cassandra/pull/2282#issuecomment-1609635459
the `class` change is good. it would be great to see a test around these settings being applied on a table at creation time, and then that changing the yaml settings never interferes (even if you recreate the table from its DDL). there's a few variations here, and the consequences can be bad. -- 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: pr-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For additional commands, e-mail: pr-h...@cassandra.apache.org