search

cmu-db / peloton

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

Update update_test.cpp #1393

Closed Traveler98 closed 6 years ago

Traveler98 commented 6 years ago

Changed LOG_INFO to LOG_DEBUG

tli2 commented 6 years ago

Why is this relevant?

Traveler98 commented 6 years ago

I am new to this, maybe i added this for no reason! But there was a post asking for this change. I appreciate your help if i was wrong.

tli2 commented 6 years ago

You mean #1392

tli2 commented 6 years ago

Okay, it doesn't make sense to apply this fix to this test case because there's another issue with it: It doesn't have any assertions in it. If you still would like to work on this issue, do you mind replacing all the print out messages with EXPECTs instead? (A good example would be, say CreateTest)