Disallow id field for batch counting
The following discussion from !28518 (merged) should be addressed:
-
@alinamihaila started a discussion: @a_akgun nice improvement, thank you!
Do you think we can have in a different MR a check for distinct_count usage and :id column, as count (distinct id)
I remember I noticed at some point in an MR
distinct_count(relation, :id)
if we add a check in distinct_count method and raise an error if column is id I think it could help.
Just a suggestion for a different MR, let me know what do you think