Re: Add SPLIT PARTITION/MERGE PARTITIONS commands

Hi, Alexander!

Thank you for your corrections!
Two questions:
----

1) "v65-0001-Implement-ALTER-TABLE-.-MERGE-PARTITIONS-.-comma.patch"
patch, "src/backend/partitioning/partbounds.c" file,
"check_two_partitions_bounds_range" function.

Old comment (v64)

+ /*
+ * lower1=false (the second to last argument) for correct comparison of
+ * lower and upper bounds.
+ */

was changed to (v65)

+ /*
+ * cmpval == false for the correct comparison result of the lower and
+ * upper bounds.
+ */

Maybe it's better to keep the old comment (or keep its meaning)?
----

2) "v65-0002-Implement-ALTER-TABLE-.-SPLIT-PARTITION-.-comman.patch"
patch, "src/backend/parser/parse_utilcmd.c" file, "checkPartition" function.

The third argument of function was changed from "is_merge" (v64) to
"isMerge" (v65). Maybe the function description should be changed in
the same way?

+ * is_merge: true indicates the operation is "ALTER TABLE ... MERGE
PARTITIONS";

--
With best regards,
Dmitry Koval

Postgres Professional: http://postgrespro.com