palantir / cassandra

Palantir's fork of Apache Cassandra
Apache License 2.0
10 stars 7 forks source link

Prohibit counter columns #563

Closed wi11dey closed 1 month ago

wi11dey commented 1 month ago

Having this precondition will allow us to assume that it is always safe to keep all inputs to a compaction

tpetracca commented 2 weeks ago

should we also fail on attempts to modify the schema at runtime which introduces a counter column? this PR only touches validating the existing schema on startup.

wi11dey commented 2 weeks ago

yeah good idea, this one was to ensure nobody was currently using counter columns

done https://github.com/palantir/cassandra/pull/572