Fork me on GitHub
#datomic
<
2021-01-27
>
jaret17:01:56

CCing @favila as we made a change to attribute predicates per the support case you logged awhile back ^

favila17:01:50

Wow that is an incredible relief. I was pretty sure when that ticket closed that you would go the other way and assert on all retractions, including the ones you missed

favila17:01:43

Many thanks, this makes our lives much easier

jaret17:01:05

There was a lot of discussion around this. Please let us know after you have used it in anger your thoughts.

favila17:01:44

I will tell you that having had to hammer existing values to fit into a reasonable predicate has not been fun

jaret17:01:52

Sorry about that. Hopefully this works much better.

👍 3
JohnJ01:01:30

attribute predicates were applied on retractions too? or what was the issue here?

favila15:01:34

Yes, they were applied on retractions too, mostly

JohnJ15:01:18

what problems did it cause?

favila15:01:34

If you have existing data that violates a predicate, you can’t add the predicate you want and then get rid of bad data. You have to either compromise the predicate to fit the bad data you have (possibly allowing new bad data); or you have to get rid of the bad data first, install the predicate, then check your data again (retractions of bad data may fail in the meantime), and possibly remove the predicate and repeat.

JohnJ16:01:34

got it, thx

JohnJ16:01:34

I wonder why they didn't took the approach of "all data must satisfy the predicate before the attribute predicate can be added".

favila16:01:50

That would be a (potentially long) blocking transaction

JohnJ17:01:43

If you don't mind, besides HA, does clubhouse runs more than one transactor to serve customers?

favila17:01:16

only one, for now. We’re working on sharding.

👍 3
jaret17:01:38

Thanks for pointing that issue out to us!