Including indexes altertype bugfix

Tests for BUG PGPRO-818

Author: Mikhail Samoylov

src/backend/commands/indexcmds.c

@@ -118,6 +118,7 @@ bool
 CheckIndexCompatible(Oid oldId,
 					 char *accessMethodName,
 					 List *attributeList,
+					 List *includingattributeList,
 					 List *exclusionOpNames)
 {
 	bool		isconstraint;
@@ -134,6 +135,7 @@ CheckIndexCompatible(Oid oldId,
 	int16	   *coloptions;
 	IndexInfo  *indexInfo;
 	int			numberOfAttributes;
+	int			numberOfKeyAttributes;
 	int			old_natts;
 	bool		isnull;
 	bool		ret = true;
@@ -153,6 +155,9 @@ CheckIndexCompatible(Oid oldId,
 	isconstraint = false;
 
 	numberOfAttributes = list_length(attributeList);
+	numberOfKeyAttributes = numberOfAttributes;
+	if (includingattributeList != NULL)
+		numberOfKeyAttributes = list_length(attributeList) - list_length(includingattributeList);
 	Assert(numberOfAttributes > 0);
 	Assert(numberOfAttributes <= INDEX_MAX_KEYS);
 
@@ -178,6 +183,8 @@ CheckIndexCompatible(Oid oldId,
 	 * later on, and it would have failed then anyway.
 	 */
 	indexInfo = makeNode(IndexInfo);
+	indexInfo->ii_NumIndexKeyAttrs = numberOfKeyAttributes;
+	indexInfo->ii_NumIndexAttrs = numberOfAttributes;
 	indexInfo->ii_Expressions = NIL;
 	indexInfo->ii_ExpressionsState = NIL;
 	indexInfo->ii_PredicateState = NIL;

src/backend/commands/tablecmds.c

@@ -9009,6 +9009,7 @@ TryReuseIndex(Oid oldId, IndexStmt *stmt)
 	if (CheckIndexCompatible(oldId,
 							 stmt->accessMethod,
 							 stmt->indexParams,
+							 stmt->indexIncludingParams,
 							 stmt->excludeOpNames))
 	{
 		Relation	irel = index_open(oldId, NoLock);

src/include/commands/defrem.h

@@ -40,6 +40,7 @@ extern char *ChooseRelationName(const char *name1, const char *name2,
 extern bool CheckIndexCompatible(Oid oldId,
 					 char *accessMethodName,
 					 List *attributeList,
+					 List *includingattributeList,
 					 List *exclusionOpNames);
 extern Oid	GetDefaultOpClass(Oid type_id, Oid am_id);
 

src/test/regress/expected/alter_table.out

@@ -3016,12 +3016,12 @@ Table "public.test_add_column"
  c4     | integer | 
 
 DROP TABLE test_add_column;
-
+-- test CHANGE COLUMN SIZE WITH BTREE INDEX
 CREATE TABLE test (test_column CHARACTER VARYING(128));
 CREATE INDEX test_index ON test USING btree(upper(test_column));
 ALTER TABLE test ALTER COLUMN test_column TYPE VARCHAR(2048);
 DROP TABLE test;
-
+-- test CHANGE COLUMN SIZE WITH HASH INDEX
 CREATE TABLE test (test_column CHARACTER VARYING(128));
 CREATE INDEX test_index ON test USING HASH(upper(test_column));
 WARNING:  hash indexes are not WAL-logged and their use is discouraged

src/test/regress/sql/alter_table.sql

@@ -1915,7 +1915,7 @@ DROP TABLE test_add_column;
 
 -- test CHANGE COLUMN SIZE WITH BTREE INDEX
 CREATE TABLE test (test_column CHARACTER VARYING(128));
-CREATE INDEX test_index ON test USING btree(upper(test_column);
+CREATE INDEX test_index ON test USING btree(upper(test_column));
 ALTER TABLE test ALTER COLUMN test_column TYPE VARCHAR(2048);
 DROP TABLE test;