peloton icon indicating copy to clipboard operation
peloton copied to clipboard

[15721] Compiled Catalog Query Access

Open nwang57 opened this issue 7 years ago • 2 comments

We enabled complied catalog lookup in the first pull request. Based upon this, we further support complied insert and delete queries for all catalogs. We also fix bugs for the following issues:

  • #1298 Need to manually bind the tuple value expression so that equality checks will work correctly.

  • Sequential scan assumes that the output columns should start at offset 0 otherwise PerformBinding will not correctly find the corresponding column attributes.

  • ZoneMapCatalog needs special care to avoid chicken and egg problem. Each sequential plan needs to check the zone map to know whether to scan the tile or not, but checking the zone map requires ZoneMap catalog access which leads to an infinity loop. So we ensure in the ZoneMap catalog manager that if the scanning table is ZoneMap catalog then it will just scan it without checking the ZoneMap

nwang57 avatar May 04 '18 15:05 nwang57

Coverage Status

Coverage increased (+0.04%) to 77.595% when pulling 89161d8535133892008c42d7695becd4f3e4a837 on nwang57:pcq-cp2 into 5686479c5f33087031e1b68b7832245c7886b712 on cmu-db:master.

coveralls avatar May 18 '18 05:05 coveralls

I am reviving this PR. We will need this when we get rid of the interpreted engine. It also has bug fix for #1362

apavlo avatar Jun 21 '18 13:06 apavlo