Commitc3b00db

committed

fixed case sensitive attribute name

1 parent92f1126 commitc3b00dbCopy full SHA for c3b00db

File tree

+10

-5

lines changed

+10

-5

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ INSERT INTO test.hash_rel VALUES (3, 3);`
`11`	`11`	`SELECT pathman.create_hash_partitions('test.hash_rel', 'value', 3);`
`12`	`12`	`ERROR: Partitioning key 'value' must be NOT NULL`
`13`	`13`	`ALTER TABLE test.hash_rel ALTER COLUMN value SET NOT NULL;`
`14`		`-SELECT pathman.create_hash_partitions('test.hash_rel', 'value', 3);`
	`14`	`+SELECT pathman.create_hash_partitions('test.hash_rel', 'Value', 3);`
`15`	`15`	`NOTICE: function test.hash_rel_hash_insert_trigger_func() does not exist, skipping`
`16`	`16`	`NOTICE: function test.hash_rel_hash_update_trigger_func() does not exist, skipping`
`17`	`17`	`NOTICE: Copying data to partitions...`
`@@ -59,7 +59,7 @@ ERROR: Partitioning key 'dt' must be NOT NULL P0001`
`59`	`59`	`ALTER TABLE test.range_rel ALTER COLUMN dt SET NOT NULL;`
`60`	`60`	`SELECT pathman.create_range_partitions('test.range_rel', 'dt', '2015-01-01'::DATE, '1 month'::INTERVAL, 2);`
`61`	`61`	`ERROR: Not enough partitions to fit all the values of 'dt' P0001`
`62`		`-SELECT pathman.create_range_partitions('test.range_rel', 'dt', '2015-01-01'::DATE, '1 month'::INTERVAL);`
	`62`	`+SELECT pathman.create_range_partitions('test.range_rel', 'DT', '2015-01-01'::DATE, '1 month'::INTERVAL);`
`63`	`63`	`NOTICE: sequence "range_rel_seq" does not exist, skipping`
`64`	`64`	`NOTICE: Copying data to partitions...`
`65`	`65`	`create_range_partitions`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,7 @@ DECLARE`
`21`	`21`	`v_typeTEXT;`
`22`	`22`	`BEGIN`
`23`	`23`	`relation := @extschema@.validate_relname(relation);`
	`24`	`+attribute :=lower(attribute);`
`24`	`25`	`PERFORM @extschema@.common_relation_checks(relation, attribute);`
`25`	`26`
`26`	`27`	`v_type := @extschema@.get_attribute_type_name(relation, attribute);`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,7 @@ load_relations_hashtable(bool reinitialize)`
`175`	`175`	`charsql[]="SELECT pg_class.relfilenode, pg_attribute.attnum, cfg.parttype, pg_attribute.atttypid "`
`176`	`176`	`"FROM %s.pathman_config as cfg "`
`177`	`177`	`"JOIN pg_class ON pg_class.relfilenode = cfg.relname::regclass::oid "`
`178`		`-"JOIN pg_attribute ON pg_attribute.attname = cfg.attname "`
	`178`	`+"JOIN pg_attribute ON pg_attribute.attname =lower(cfg.attname) "`
`179`	`179`	`"AND attrelid = pg_class.relfilenode";`
`180`	`180`	`char*query;`
`181`	`181`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,7 @@ DECLARE`
`26`	`26`	`iINTEGER;`
`27`	`27`	`BEGIN`
`28`	`28`	`p_relation := @extschema@.validate_relname(p_relation);`
	`29`	`+p_attribute :=lower(p_attribute);`
`29`	`30`	`PERFORM @extschema@.common_relation_checks(p_relation, p_attribute);`
`30`	`31`
`31`	`32`	`/* Try to determine partitions count if not set*/`
`@@ -110,6 +111,7 @@ DECLARE`
`110`	`111`	`iINTEGER;`
`111`	`112`	`BEGIN`
`112`	`113`	`p_relation := @extschema@.validate_relname(p_relation);`
	`114`	`+p_attribute :=lower(p_attribute);`
`113`	`115`	`PERFORM @extschema@.common_relation_checks(p_relation, p_attribute);`
`114`	`116`
`115`	`117`	`IF p_count<=0 THEN`
`@@ -195,6 +197,7 @@ DECLARE`
`195`	`197`	`iINTEGER :=0;`
`196`	`198`	`BEGIN`
`197`	`199`	`p_relation := @extschema@.validate_relname(p_relation);`
	`200`	`+p_attribute :=lower(p_attribute);`
`198`	`201`	`PERFORM @extschema@.common_relation_checks(p_relation, p_attribute);`
`199`	`202`
`200`	`203`	`IF p_interval<=0 THEN`
`@@ -258,6 +261,7 @@ DECLARE`
`258`	`261`	`iINTEGER :=0;`
`259`	`262`	`BEGIN`
`260`	`263`	`p_relation := @extschema@.validate_relname(p_relation);`
	`264`	`+p_attribute :=lower(p_attribute);`
`261`	`265`	`PERFORM @extschema@.common_relation_checks(p_relation, p_attribute);`
`262`	`266`
`263`	`267`	`EXECUTE format('DROP SEQUENCE IF EXISTS %s_seq', p_relation);`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ INSERT INTO test.hash_rel VALUES (2, 2);`
`12`	`12`	`INSERT INTOtest.hash_relVALUES (3,3);`
`13`	`13`	`SELECTpathman.create_hash_partitions('test.hash_rel','value',3);`
`14`	`14`	`ALTERTABLEtest.hash_rel ALTER COLUMN valueSETNOT NULL;`
`15`		`-SELECTpathman.create_hash_partitions('test.hash_rel','value',3);`
	`15`	`+SELECTpathman.create_hash_partitions('test.hash_rel','Value',3);`
`16`	`16`	`SELECTCOUNT(*)FROMtest.hash_rel;`
`17`	`17`	`SELECTCOUNT(*)FROM ONLYtest.hash_rel;`
`18`	`18`	`INSERT INTOtest.hash_relVALUES (4,4);`
`@@ -31,7 +31,7 @@ SELECT g, md5(g::TEXT) FROM generate_series('2015-01-01', '2015-04-30', '1 day':`
`31`	`31`	`SELECTpathman.create_range_partitions('test.range_rel','dt','2015-01-01'::DATE,'1 month'::INTERVAL,2);`
`32`	`32`	`ALTERTABLEtest.range_rel ALTER COLUMN dtSETNOT NULL;`
`33`	`33`	`SELECTpathman.create_range_partitions('test.range_rel','dt','2015-01-01'::DATE,'1 month'::INTERVAL,2);`
`34`		`-SELECTpathman.create_range_partitions('test.range_rel','dt','2015-01-01'::DATE,'1 month'::INTERVAL);`
	`34`	`+SELECTpathman.create_range_partitions('test.range_rel','DT','2015-01-01'::DATE,'1 month'::INTERVAL);`
`35`	`35`	`SELECTCOUNT(*)FROMtest.range_rel;`
`36`	`36`	`SELECTCOUNT(*)FROM ONLYtest.range_rel;`
`37`	`37`

Comments

(0)