Skip to content
Snippets Groups Projects
  • Tom Lane's avatar
    53299429
    Revert patch for taking fewer snapshots. · 53299429
    Tom Lane authored
    This reverts commit d573e239, "Take fewer
    snapshots".  While that seemed like a good idea at the time, it caused
    execution to use a snapshot that had been acquired before locking any of
    the tables mentioned in the query.  This created user-visible anomalies
    that were not present in any prior release of Postgres, as reported by
    Tomas Vondra.  While this whole area could do with a redesign (since there
    are related cases that have anomalies anyway), it doesn't seem likely that
    any future patch would be reasonably back-patchable; and we don't want 9.2
    to exhibit a behavior that's subtly unlike either past or future releases.
    Hence, revert to prior code while we rethink the problem.
    53299429
    History
    Revert patch for taking fewer snapshots.
    Tom Lane authored
    This reverts commit d573e239, "Take fewer
    snapshots".  While that seemed like a good idea at the time, it caused
    execution to use a snapshot that had been acquired before locking any of
    the tables mentioned in the query.  This created user-visible anomalies
    that were not present in any prior release of Postgres, as reported by
    Tomas Vondra.  While this whole area could do with a redesign (since there
    are related cases that have anomalies anyway), it doesn't seem likely that
    any future patch would be reasonably back-patchable; and we don't want 9.2
    to exhibit a behavior that's subtly unlike either past or future releases.
    Hence, revert to prior code while we rethink the problem.