diff options
| author | Robert Haas <rhaas@postgresql.org> | 2013-07-23 10:58:32 -0400 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2013-07-23 10:58:32 -0400 |
| commit | 765ad89be36f699e2d65238c1f0458a1ab7e4d8b (patch) | |
| tree | 51a0157a5efb3767283f2bfb183c1ac6d3535652 /src | |
| parent | e6055061c524698918ab0b7a0c51b822c03ef1fa (diff) | |
| download | postgresql-765ad89be36f699e2d65238c1f0458a1ab7e4d8b.tar.gz postgresql-765ad89be36f699e2d65238c1f0458a1ab7e4d8b.zip | |
Use InvalidSnapshot, now SnapshotNow, as the default snapshot.
As far as I can determine, there's no code in the core distribution
that fails to explicitly set the snapshot of a scan or executor
state. If there is any such code, this will probably cause it to
seg fault; friendlier suggestions were discussed on pgsql-hackers,
but there was no consensus that anything more than this was
needed.
This is another step towards the hoped-for complete removal of
SnapshotNow.
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/access/index/genam.c | 2 | ||||
| -rw-r--r-- | src/backend/executor/execUtils.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/access/index/genam.c b/src/backend/access/index/genam.c index 60a2b4e4389..15debedf406 100644 --- a/src/backend/access/index/genam.c +++ b/src/backend/access/index/genam.c @@ -79,7 +79,7 @@ RelationGetIndexScan(Relation indexRelation, int nkeys, int norderbys) scan->heapRelation = NULL; /* may be set later */ scan->indexRelation = indexRelation; - scan->xs_snapshot = SnapshotNow; /* may be set later */ + scan->xs_snapshot = InvalidSnapshot; /* caller must initialize this */ scan->numberOfKeys = nkeys; scan->numberOfOrderBys = norderbys; diff --git a/src/backend/executor/execUtils.c b/src/backend/executor/execUtils.c index b449e0a5534..39e3b2e79c8 100644 --- a/src/backend/executor/execUtils.c +++ b/src/backend/executor/execUtils.c @@ -105,7 +105,7 @@ CreateExecutorState(void) * Initialize all fields of the Executor State structure */ estate->es_direction = ForwardScanDirection; - estate->es_snapshot = SnapshotNow; + estate->es_snapshot = InvalidSnapshot; /* caller must initialize this */ estate->es_crosscheck_snapshot = InvalidSnapshot; /* no crosscheck */ estate->es_range_table = NIL; estate->es_plannedstmt = NULL; |