citus/src
Jason Petersen 37d7f98f50 Add clarifying comment in HashableClauseMutator
While reading this code last week, it appeared as though there was no
place we ensured that the partition clause actually used equality ops.
As such, I was worried that we might transform a clause such as id < 5
into a constraint like hash(id) = hash(5) when doing shard pruning. The
relevant code seemed to just ensure:

  1. The node is an OpExpr
  2. With a related hash function
  3. It compares the partition column
  4. Against a constant

A superficial reading implied we didn't actually make sure the original
op was equality-related, but it turns out the hash lookup function DOES
ensure that for us. So I added a comment.
2016-04-19 12:21:11 -06:00
..
backend Add clarifying comment in HashableClauseMutator 2016-04-19 12:21:11 -06:00
bin Update copyright dates 2016-03-23 17:14:37 -06:00
include Fix use of INT64CONST macro 2016-04-15 12:13:56 -06:00
test/regress Better error on "CREATE INDEX already_exists ..." 2016-04-18 13:33:53 -07:00