Move tile group related code from catalog manager to storage manager

cmu-db / peloton

The Self-Driving Database Management System

http://pelotondb.io

Apache License 2.0

2.03k stars 623 forks source link

Move tile group related code from catalog manager to storage manager #1376

Closed db-ol closed 6 years ago

poojanilangekar commented 6 years ago

@aaron-tian. You will have to refactor this PR. After merging #1375, we now use CuckooMap for the tile_group_locator_.

coveralls commented 6 years ago

Coverage decreased (-0.2%) to 77.364% when pulling bcc6c02d8c0f8c4edad8e798ca1f5cffd76beca3 on aaron-tian:quick_fix into 5686479c5f33087031e1b68b7832245c7886b712 on cmu-db:master.

apavlo commented 6 years ago

@aaron-tian You also have unnecessary duplicated commits. Start with a clean branch and redo this.

db-ol commented 6 years ago

@apavlo Thanks for reminding us. Ao introduced them accidentally. He will manually copy the code and make a new PR.