Commit 6d08b6a7 authored by Bruce Momjian's avatar Bruce Momjian

Remove 4096 string limited key on block size

parent 0a8fb5a8
...@@ -10,7 +10,7 @@ ...@@ -10,7 +10,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/parser/gram.y,v 2.77 1999/05/17 01:01:06 momjian Exp $ * $Header: /cvsroot/pgsql/src/backend/parser/gram.y,v 2.78 1999/05/19 17:53:10 momjian Exp $
* *
* HISTORY * HISTORY
* AUTHOR DATE MAJOR EVENT * AUTHOR DATE MAJOR EVENT
...@@ -3403,13 +3403,8 @@ Character: character '(' Iconst ')' ...@@ -3403,13 +3403,8 @@ Character: character '(' Iconst ')'
if ($3 < 1) if ($3 < 1)
elog(ERROR,"length for '%s' type must be at least 1",$1); elog(ERROR,"length for '%s' type must be at least 1",$1);
else if ($3 > 4096) else if ($3 > BLCKSZ - 128)
/* we can store a char() of length up to the size elog(ERROR,"length for type '%s' cannot exceed %d",$1, BLCKSZ-128);
* of a page (8KB) - page headers and friends but
* just to be safe here... - ay 6/95
* XXX note this hardcoded limit - thomas 1997-07-13
*/
elog(ERROR,"length for type '%s' cannot exceed 4096",$1);
/* we actually implement this sort of like a varlen, so /* we actually implement this sort of like a varlen, so
* the first 4 bytes is the length. (the difference * the first 4 bytes is the length. (the difference
......
...@@ -7,7 +7,7 @@ ...@@ -7,7 +7,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/utils/adt/varchar.c,v 1.44 1999/05/03 19:10:02 momjian Exp $ * $Header: /cvsroot/pgsql/src/backend/utils/adt/varchar.c,v 1.45 1999/05/19 17:53:11 momjian Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
...@@ -81,8 +81,8 @@ bpcharin(char *s, int dummy, int32 atttypmod) ...@@ -81,8 +81,8 @@ bpcharin(char *s, int dummy, int32 atttypmod)
else else
len = atttypmod - VARHDRSZ; len = atttypmod - VARHDRSZ;
if (len > 4096) if (len > BLCKSZ - 128)
elog(ERROR, "bpcharin: length of char() must be less than 4096"); elog(ERROR, "bpcharin: length of char() must be less than %d",BLCKSZ-128);
result = (char *) palloc(atttypmod); result = (char *) palloc(atttypmod);
VARSIZE(result) = atttypmod; VARSIZE(result) = atttypmod;
...@@ -151,8 +151,8 @@ bpchar(char *s, int32 len) ...@@ -151,8 +151,8 @@ bpchar(char *s, int32 len)
rlen = len - VARHDRSZ; rlen = len - VARHDRSZ;
if (rlen > 4096) if (rlen > BLCKSZ - 128)
elog(ERROR, "bpchar: length of char() must be less than 4096"); elog(ERROR, "bpchar: length of char() must be less than %d",BLCKSZ-128);
#ifdef STRINGDEBUG #ifdef STRINGDEBUG
printf("bpchar- convert string length %d (%d) ->%d (%d)\n", printf("bpchar- convert string length %d (%d) ->%d (%d)\n",
...@@ -330,8 +330,8 @@ varcharin(char *s, int dummy, int32 atttypmod) ...@@ -330,8 +330,8 @@ varcharin(char *s, int dummy, int32 atttypmod)
if (atttypmod != -1 && len > atttypmod) if (atttypmod != -1 && len > atttypmod)
len = atttypmod; /* clip the string at max length */ len = atttypmod; /* clip the string at max length */
if (len > 4096) if (len > BLCKSZ - 128)
elog(ERROR, "varcharin: length of char() must be less than 4096"); elog(ERROR, "varcharin: length of char() must be less than %d",BLCKSZ-128);
result = (char *) palloc(len); result = (char *) palloc(len);
VARSIZE(result) = len; VARSIZE(result) = len;
...@@ -398,8 +398,8 @@ varchar(char *s, int32 slen) ...@@ -398,8 +398,8 @@ varchar(char *s, int32 slen)
len = slen - VARHDRSZ; len = slen - VARHDRSZ;
#endif #endif
if (len > 4096) if (len > BLCKSZ-128)
elog(ERROR, "varchar: length of varchar() must be less than 4096"); elog(ERROR, "varchar: length of varchar() must be less than BLCKSZ-128");
result = (char *) palloc(slen); result = (char *) palloc(slen);
VARSIZE(result) = slen; VARSIZE(result) = slen;
......
...@@ -3355,13 +3355,12 @@ Character: character '(' Iconst ')' ...@@ -3355,13 +3355,12 @@ Character: character '(' Iconst ')'
sprintf(errortext, "length for '%s' type must be at least 1",$1); sprintf(errortext, "length for '%s' type must be at least 1",$1);
yyerror(errortext); yyerror(errortext);
} }
else if (atol($3) > 4096) { else if (atol($3) > BLCKSZ - 128) {
/* we can store a char() of length up to the size /* we can store a char() of length up to the size
* of a page (8KB) - page headers and friends but * of a page (8KB) - page headers and friends but
* just to be safe here... - ay 6/95 * just to be safe here... - ay 6/95
* XXX note this hardcoded limit - thomas 1997-07-13
*/ */
sprintf(errortext, "length for type '%s' cannot exceed 4096",$1); sprintf(errortext, "length for type '%s' cannot exceed %d",BLCKSZ-128);
yyerror(errortext); yyerror(errortext);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment