don-tay / simpledb

Subset of SQL-compliant DBMS
0 stars 0 forks source link

Fix MergeJoinScan next logic #25

Closed tishyakhanna97 closed 2 years ago

tishyakhanna97 commented 2 years ago

Changes in this PR

Overview

We were skipping the "next" call in the LHS table, causing the pointer on the LHS table to remain there permanently. This caused an infinite loop, which can be seen here https://github.com/don-tay/simpledb/issues/23