search

cmu-db / peloton

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

IndexScanDesc and IndexScanPlan Refactoring #1301

Closed linmagit closed 6 years ago

linmagit commented 6 years ago

This PR fixes the issue in #1300. It takes the pointer out of the IndexScanDesc and IndexScanPlan, and replace them with the index OID. The IndexScanExecutor will get the physical pointer to the index object in its DInit() function.

This PR also adjusts the calls to the IndexScanDesc constructor in many test cases.

linmagit commented 6 years ago

@vkonagar @sivaprasadsudhir @pbollimp Let me know whether this satisfies your needs.

apavlo commented 6 years ago

I will merge this when the build passes.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.006%) to 77.267% when pulling 6fe7a2e8822f486c4ed8e77f94abf971bd59402f on malin1993ml:indexscanplan-fix into d583b5f18e9853e7074ab99bcfd1e8ebc23a9a35 on cmu-db:master.