Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit1c53c4d

Browse files
committed
Finish reverting "recheck_on_update" patch.
This reverts commitc203d6c and some follow-on fixes, completing thetask begun in commit5d28c9b. If that feature is ever resurrected,the code will look quite a bit different from this, so it seems bestto start from a clean slate.The v11 branch is not touched; in that branch, the recheck_on_updatestorage option remains present, but nonfunctional and undocumented.Discussion:https://postgr.es/m/20190114223409.3tcvejfhlvbucrv5@alap3.anarazel.de
1 parent0944ec5 commit1c53c4d

File tree

8 files changed

+21
-277
lines changed

8 files changed

+21
-277
lines changed

‎src/backend/access/common/reloptions.c

Lines changed: 1 addition & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -129,15 +129,6 @@ static relopt_bool boolRelOpts[] =
129129
},
130130
true
131131
},
132-
{
133-
{
134-
"recheck_on_update",
135-
"Recheck functional index expression for changed value after update",
136-
RELOPT_KIND_INDEX,
137-
ShareUpdateExclusiveLock/* since only applies to later UPDATEs */
138-
},
139-
true
140-
},
141132
{
142133
{
143134
"security_barrier",
@@ -1343,7 +1334,7 @@ fillRelOptions(void *rdopts, Size basesize,
13431334
break;
13441335
}
13451336
}
1346-
if (validate&& !found&&options[i].gen->kinds!=RELOPT_KIND_INDEX)
1337+
if (validate&& !found)
13471338
elog(ERROR,"reloption \"%s\" not found in parse table",
13481339
options[i].gen->name);
13491340
}
@@ -1501,40 +1492,6 @@ index_reloptions(amoptions_function amoptions, Datum reloptions, bool validate)
15011492
returnamoptions(reloptions,validate);
15021493
}
15031494

1504-
/*
1505-
* Parse generic options for all indexes.
1506-
*
1507-
*reloptionsoptions as text[] datum
1508-
*validateerror flag
1509-
*/
1510-
bytea*
1511-
index_generic_reloptions(Datumreloptions,boolvalidate)
1512-
{
1513-
intnumoptions;
1514-
GenericIndexOpts*idxopts;
1515-
relopt_value*options;
1516-
staticconstrelopt_parse_elttab[]= {
1517-
{"recheck_on_update",RELOPT_TYPE_BOOL, offsetof(GenericIndexOpts,recheck_on_update)}
1518-
};
1519-
1520-
options=parseRelOptions(reloptions,validate,
1521-
RELOPT_KIND_INDEX,
1522-
&numoptions);
1523-
1524-
/* if none set, we're done */
1525-
if (numoptions==0)
1526-
returnNULL;
1527-
1528-
idxopts=allocateReloptStruct(sizeof(GenericIndexOpts),options,numoptions);
1529-
1530-
fillRelOptions((void*)idxopts,sizeof(GenericIndexOpts),options,numoptions,
1531-
validate,tab,lengthof(tab));
1532-
1533-
pfree(options);
1534-
1535-
return (bytea*)idxopts;
1536-
}
1537-
15381495
/*
15391496
* Option parser for attribute reloptions
15401497
*/

‎src/backend/access/heap/heapam.c

Lines changed: 7 additions & 102 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,6 @@
5757
#include"access/xlogutils.h"
5858
#include"catalog/catalog.h"
5959
#include"catalog/namespace.h"
60-
#include"catalog/index.h"
6160
#include"miscadmin.h"
6261
#include"pgstat.h"
6362
#include"port/atomics.h"
@@ -76,9 +75,7 @@
7675
#include"utils/snapmgr.h"
7776
#include"utils/syscache.h"
7877
#include"utils/tqual.h"
79-
#include"utils/memutils.h"
80-
#include"nodes/execnodes.h"
81-
#include"executor/executor.h"
78+
8279

8380
/* GUC variable */
8481
boolsynchronize_seqscans= true;
@@ -130,7 +127,6 @@ static bool ConditionalMultiXactIdWait(MultiXactId multi, MultiXactStatus status
130127
staticXLogRecPtrlog_heap_new_cid(Relationrelation,HeapTupletup);
131128
staticHeapTupleExtractReplicaIdentity(Relationrel,HeapTupletup,boolkey_modified,
132129
bool*copy);
133-
staticboolProjIndexIsUnchanged(Relationrelation,HeapTupleoldtup,HeapTuplenewtup);
134130

135131

136132
/*
@@ -3511,7 +3507,6 @@ heap_update(Relation relation, ItemPointer otid, HeapTuple newtup,
35113507
HTSU_Resultresult;
35123508
TransactionIdxid=GetCurrentTransactionId();
35133509
Bitmapset*hot_attrs;
3514-
Bitmapset*proj_idx_attrs;
35153510
Bitmapset*key_attrs;
35163511
Bitmapset*id_attrs;
35173512
Bitmapset*interesting_attrs;
@@ -3575,11 +3570,12 @@ heap_update(Relation relation, ItemPointer otid, HeapTuple newtup,
35753570
* Note that we get copies of each bitmap, so we need not worry about
35763571
* relcache flush happening midway through.
35773572
*/
3578-
hot_attrs=RelationGetIndexAttrBitmap(relation,INDEX_ATTR_BITMAP_HOT);
3579-
proj_idx_attrs=RelationGetIndexAttrBitmap(relation,INDEX_ATTR_BITMAP_PROJ);
3573+
hot_attrs=RelationGetIndexAttrBitmap(relation,INDEX_ATTR_BITMAP_ALL);
35803574
key_attrs=RelationGetIndexAttrBitmap(relation,INDEX_ATTR_BITMAP_KEY);
35813575
id_attrs=RelationGetIndexAttrBitmap(relation,
35823576
INDEX_ATTR_BITMAP_IDENTITY_KEY);
3577+
3578+
35833579
block=ItemPointerGetBlockNumber(otid);
35843580
buffer=ReadBuffer(relation,block);
35853581
page=BufferGetPage(buffer);
@@ -3599,7 +3595,6 @@ heap_update(Relation relation, ItemPointer otid, HeapTuple newtup,
35993595
if (!PageIsFull(page))
36003596
{
36013597
interesting_attrs=bms_add_members(interesting_attrs,hot_attrs);
3602-
interesting_attrs=bms_add_members(interesting_attrs,proj_idx_attrs);
36033598
hot_attrs_checked= true;
36043599
}
36053600
interesting_attrs=bms_add_members(interesting_attrs,key_attrs);
@@ -3883,7 +3878,6 @@ heap_update(Relation relation, ItemPointer otid, HeapTuple newtup,
38833878
if (vmbuffer!=InvalidBuffer)
38843879
ReleaseBuffer(vmbuffer);
38853880
bms_free(hot_attrs);
3886-
bms_free(proj_idx_attrs);
38873881
bms_free(key_attrs);
38883882
bms_free(id_attrs);
38893883
bms_free(modified_attrs);
@@ -4191,18 +4185,11 @@ heap_update(Relation relation, ItemPointer otid, HeapTuple newtup,
41914185
/*
41924186
* Since the new tuple is going into the same page, we might be able
41934187
* to do a HOT update. Check if any of the index columns have been
4194-
* changed, or if we have projection functional indexes, check whether
4195-
* the old and the new values are the same. If the page was already
4196-
* full, we may have skipped checking for index columns. If so, HOT
4197-
* update is possible.
4188+
* changed. If the page was already full, we may have skipped checking
4189+
* for index columns. If so, HOT update is possible.
41984190
*/
4199-
if (hot_attrs_checked
4200-
&& !bms_overlap(modified_attrs,hot_attrs)
4201-
&& (!bms_overlap(modified_attrs,proj_idx_attrs)
4202-
||ProjIndexIsUnchanged(relation,&oldtup,newtup)))
4203-
{
4191+
if (hot_attrs_checked&& !bms_overlap(modified_attrs,hot_attrs))
42044192
use_hot_update= true;
4205-
}
42064193
}
42074194
else
42084195
{
@@ -4364,7 +4351,6 @@ heap_update(Relation relation, ItemPointer otid, HeapTuple newtup,
43644351
heap_freetuple(old_key_tuple);
43654352

43664353
bms_free(hot_attrs);
4367-
bms_free(proj_idx_attrs);
43684354
bms_free(key_attrs);
43694355
bms_free(id_attrs);
43704356
bms_free(modified_attrs);
@@ -4450,87 +4436,6 @@ heap_tuple_attr_equals(TupleDesc tupdesc, int attrnum,
44504436
}
44514437
}
44524438

4453-
/*
4454-
* Check whether the value is unchanged after update of a projection
4455-
* functional index. Compare the new and old values of the indexed
4456-
* expression to see if we are able to use a HOT update or not.
4457-
*/
4458-
staticbool
4459-
ProjIndexIsUnchanged(Relationrelation,HeapTupleoldtup,HeapTuplenewtup)
4460-
{
4461-
ListCell*l;
4462-
List*indexoidlist=RelationGetIndexList(relation);
4463-
EState*estate=CreateExecutorState();
4464-
ExprContext*econtext=GetPerTupleExprContext(estate);
4465-
TupleTableSlot*slot=MakeSingleTupleTableSlot(RelationGetDescr(relation),
4466-
&TTSOpsHeapTuple);
4467-
boolequals= true;
4468-
Datumold_values[INDEX_MAX_KEYS];
4469-
boolold_isnull[INDEX_MAX_KEYS];
4470-
Datumnew_values[INDEX_MAX_KEYS];
4471-
boolnew_isnull[INDEX_MAX_KEYS];
4472-
intindexno=0;
4473-
4474-
econtext->ecxt_scantuple=slot;
4475-
4476-
foreach(l,indexoidlist)
4477-
{
4478-
if (bms_is_member(indexno,relation->rd_projidx))
4479-
{
4480-
OidindexOid=lfirst_oid(l);
4481-
RelationindexDesc=index_open(indexOid,AccessShareLock);
4482-
IndexInfo*indexInfo=BuildIndexInfo(indexDesc);
4483-
inti;
4484-
4485-
ResetExprContext(econtext);
4486-
ExecStoreHeapTuple(oldtup,slot, false);
4487-
FormIndexDatum(indexInfo,
4488-
slot,
4489-
estate,
4490-
old_values,
4491-
old_isnull);
4492-
4493-
ExecStoreHeapTuple(newtup,slot, false);
4494-
FormIndexDatum(indexInfo,
4495-
slot,
4496-
estate,
4497-
new_values,
4498-
new_isnull);
4499-
4500-
for (i=0;i<indexInfo->ii_NumIndexAttrs;i++)
4501-
{
4502-
if (old_isnull[i]!=new_isnull[i])
4503-
{
4504-
equals= false;
4505-
break;
4506-
}
4507-
elseif (!old_isnull[i])
4508-
{
4509-
Form_pg_attributeatt=TupleDescAttr(RelationGetDescr(indexDesc),i);
4510-
4511-
if (!datumIsEqual(old_values[i],new_values[i],att->attbyval,att->attlen))
4512-
{
4513-
equals= false;
4514-
break;
4515-
}
4516-
}
4517-
}
4518-
index_close(indexDesc,AccessShareLock);
4519-
4520-
if (!equals)
4521-
{
4522-
break;
4523-
}
4524-
}
4525-
indexno+=1;
4526-
}
4527-
ExecDropSingleTupleTableSlot(slot);
4528-
FreeExecutorState(estate);
4529-
4530-
returnequals;
4531-
}
4532-
4533-
45344439
/*
45354440
* Check which columns are being updated.
45364441
*

‎src/backend/catalog/index.c

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@
2727
#include"access/heapam.h"
2828
#include"access/multixact.h"
2929
#include"access/relscan.h"
30-
#include"access/reloptions.h"
3130
#include"access/sysattr.h"
3231
#include"access/transam.h"
3332
#include"access/visibilitymap.h"
@@ -355,7 +354,7 @@ ConstructTupleDescriptor(Relation heapRelation,
355354
/* Simple index column */
356355
constFormData_pg_attribute*from;
357356

358-
Assert(atnum>0);/* should've been caught above */
357+
Assert(atnum>0);/* should've been caught above */
359358

360359
if (atnum>natts)/* safety check */
361360
elog(ERROR,"invalid column number %d",atnum);
@@ -3903,7 +3902,7 @@ reindex_relation(Oid relid, int flags, int options)
39033902

39043903
/* Ensure rd_indexattr is valid; see comments for RelationSetIndexList */
39053904
if (is_pg_class)
3906-
(void)RelationGetIndexAttrBitmap(rel,INDEX_ATTR_BITMAP_HOT);
3905+
(void)RelationGetIndexAttrBitmap(rel,INDEX_ATTR_BITMAP_ALL);
39073906

39083907
PG_TRY();
39093908
{

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp