Skip to content
This repository has been archived by the owner on Sep 27, 2019. It is now read-only.

[15721] Pre-compiled catalog access #1281

Closed
wants to merge 102 commits into from
Closed

Conversation

ruogudu
Copy link

@ruogudu ruogudu commented Apr 11, 2018

Replace the original interpreted index scan by pre-compiled sequential scan in catalogs with query cache.
Created new function: GetResultWithCompiledSeqScan in /catalog/abstract_catalog.cpp
Modified other catalog source files including

src/catalog/column_catalog.cpp
src/catalog/column_stats_catalog.cpp
src/catalog/database_catalog.cpp
src/catalog/index_catalog.cpp
src/catalog/language_catalog.cpp
src/catalog/proc_catalog.cpp
src/catalog/settings_catalog.cpp
src/catalog/table_catalog.cpp
src/catalog/trigger_catalog.cpp
src/catalog/zone_map_catalog.cpp

@Zeninma
Copy link
Contributor

Zeninma commented Apr 28, 2018

@apavlo Regarding your comments about the "big issue", what do you exactly mean by "copying data out of the WrappedTuple and putting into the catalog objects"? Do you suggest creating constructors taking wrapped tuple for all the catalogs and removing the old constructors with logical tiles?

@apavlo
Copy link
Member

apavlo commented Jun 21, 2018

Replaced by #1339

@apavlo apavlo closed this Jun 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants