BwTree non-unique key insert fix

cmu-db / peloton

The Self-Driving Database Management System

http://pelotondb.io

Apache License 2.0

2.04k stars 623 forks source link

BwTree non-unique key insert fix #1276

Closed wangziqi2013 closed 6 years ago

coveralls commented 6 years ago

Coverage increased (+0.04%) to 77.007% when pulling 09cf61228498eea1254dfab083098134ea272dfc on wangziqi2013:non-unique-key-fix into a9d83a04918bd8423bc218090b06bdd52291526d on cmu-db:master.

pmenon commented 6 years ago

@wangziqi2013 Can you enable the unique key tests (or add new ones) in this PR as well?

pervazea commented 6 years ago

@pmenon thanks. I was about to ask how this was to be tested. @wangziqi2013 I second pmenon's request to ensure we have a test for this.

wangziqi2013 commented 6 years ago

@pmenon @pervazea Doing it now

wangziqi2013 commented 6 years ago

@pmenon @pervazea I enabled unique key insert and delete test. The test suite is running now

pervazea commented 6 years ago

Jenkins is failing to check out your branch. Not clear why. Were there merge conflicts? Correctly merged with upstream master?