cmu-db / peloton

The Self-Driving Database Management System
http://pelotondb.io
Apache License 2.0
2.04k stars 623 forks source link

[15721] Pre-compiled catalog access #1281

Closed ruogudu closed 6 years ago

ruogudu commented 6 years ago

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
coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.6%) to 76.406% when pulling d2447ff44c755fc604b59e92327629310b7fd673 on nwang57:pcq-cp into a9d83a04918bd8423bc218090b06bdd52291526d on cmu-db:master.

Zeninma commented 6 years ago

@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 commented 6 years ago

Replaced by #1339