Skip to content
Snippets Groups Projects
Select Git revision
  • benchmark-tools
  • postgres-lambda
  • master default
  • REL9_4_25
  • REL9_5_20
  • REL9_6_16
  • REL_10_11
  • REL_11_6
  • REL_12_1
  • REL_12_0
  • REL_12_RC1
  • REL_12_BETA4
  • REL9_4_24
  • REL9_5_19
  • REL9_6_15
  • REL_10_10
  • REL_11_5
  • REL_12_BETA3
  • REL9_4_23
  • REL9_5_18
  • REL9_6_14
  • REL_10_9
  • REL_11_4
23 results

_int.sql

Blame
    • Tom Lane's avatar
      ced272d2
      Fix intarray's GiST opclasses to not fail for empty arrays with <@. · ced272d2
      Tom Lane authored
      contrib/intarray considers "arraycol <@ constant-array" to be indexable,
      but its GiST opclass code fails to reliably find index entries for empty
      array values (which of course should trivially match such queries).
      This is because the test condition to see whether we should descend
      through a non-leaf node is wrong.
      
      Unfortunately, empty array entries could be anywhere in the index,
      as these index opclasses are currently designed.  So there's no way
      to fix this except by lobotomizing <@ indexscans to scan the whole
      index ... which is what this patch does.  That's pretty unfortunate:
      the performance is now actually worse than a seqscan, in most cases.
      We'd be better off to remove <@ from the GiST opclasses entirely,
      and perhaps a future non-back-patchable patch will do so.
      
      In the meantime, applications whose performance is adversely impacted
      have a couple of options.  They could switch to a GIN index, which
      doesn't have this bug, or they could replace "arraycol <@ constant-array"
      with "arraycol <@ constant-array AND arraycol && constant-array".
      That will provide about the same performance as before, and it will find
      all non-empty subsets of the given constant-array, which is all that
      could reliably be expected of the query before.
      
      While at it, add some more regression test cases to improve code
      coverage of contrib/intarray.
      
      In passing, adjust resize_intArrayType so that when it's returning an
      empty array, it uses construct_empty_array for that rather than
      cowboy hacking on the input array.  While the hack produces an array
      that looks valid for most purposes, it isn't bitwise equal to empty
      arrays produced by other code paths, which could have subtle odd
      effects.  I don't think this code path is performance-critical
      enough to justify such shortcuts.  (Back-patch this part only as far
      as v11; before commit 01783ac3 we were not careful about this in
      other intarray code paths either.)
      
      Back-patch the <@ fixes to all supported versions, since this was
      broken from day one.
      
      Patch by me; thanks to Alexander Korotkov for review.
      
      Discussion: https://postgr.es/m/458.1565114141@sss.pgh.pa.us
      ced272d2
      History
      Fix intarray's GiST opclasses to not fail for empty arrays with <@.
      Tom Lane authored
      contrib/intarray considers "arraycol <@ constant-array" to be indexable,
      but its GiST opclass code fails to reliably find index entries for empty
      array values (which of course should trivially match such queries).
      This is because the test condition to see whether we should descend
      through a non-leaf node is wrong.
      
      Unfortunately, empty array entries could be anywhere in the index,
      as these index opclasses are currently designed.  So there's no way
      to fix this except by lobotomizing <@ indexscans to scan the whole
      index ... which is what this patch does.  That's pretty unfortunate:
      the performance is now actually worse than a seqscan, in most cases.
      We'd be better off to remove <@ from the GiST opclasses entirely,
      and perhaps a future non-back-patchable patch will do so.
      
      In the meantime, applications whose performance is adversely impacted
      have a couple of options.  They could switch to a GIN index, which
      doesn't have this bug, or they could replace "arraycol <@ constant-array"
      with "arraycol <@ constant-array AND arraycol && constant-array".
      That will provide about the same performance as before, and it will find
      all non-empty subsets of the given constant-array, which is all that
      could reliably be expected of the query before.
      
      While at it, add some more regression test cases to improve code
      coverage of contrib/intarray.
      
      In passing, adjust resize_intArrayType so that when it's returning an
      empty array, it uses construct_empty_array for that rather than
      cowboy hacking on the input array.  While the hack produces an array
      that looks valid for most purposes, it isn't bitwise equal to empty
      arrays produced by other code paths, which could have subtle odd
      effects.  I don't think this code path is performance-critical
      enough to justify such shortcuts.  (Back-patch this part only as far
      as v11; before commit 01783ac3 we were not careful about this in
      other intarray code paths either.)
      
      Back-patch the <@ fixes to all supported versions, since this was
      broken from day one.
      
      Patch by me; thanks to Alexander Korotkov for review.
      
      Discussion: https://postgr.es/m/458.1565114141@sss.pgh.pa.us