vmware-archive / quickstep

Quickstep Project
Apache License 2.0
27 stars 13 forks source link

Cleanup in Preloader class #209

Closed hbdeshmukh closed 8 years ago

hbdeshmukh commented 8 years ago

Added missing include and more informational log messages.

hbdeshmukh commented 8 years ago

Hi @pateljm

Please review this PR. It has mostly simple changes. These changes can be potentially useful in cases when we decide not to preload the complete database (e.g. for experiments involving data sets larger than buffer pool), to check which relations have been preloaded to the buffer pool.

A useful feature (TODO) will be to specify which relations can be preloaded to the buffer pool.

hbdeshmukh commented 8 years ago

Addressed all the comments.