-
Notifications
You must be signed in to change notification settings - Fork 209
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fail on init errors #4216
base: main
Are you sure you want to change the base?
Fail on init errors #4216
Conversation
c3f50f1
to
d73d901
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lint check:
@@ db/block_internal.c @@
+ Please remove debugging statements, and replace fprintf/printf with logmsg.
@@ db/handle_buf.c @@
+ Please remove debugging statements, and replace fprintf/printf with logmsg.
@@ db/sqlglue.c @@
+ Please remove debugging statements, and replace fprintf/printf with logmsg.
@@ db/osqlcomm.c @@
+ Please remove debugging statements, and replace fprintf/printf with logmsg.
@@ db/toblock.c @@
+ Please remove debugging statements, and replace fprintf/printf with logmsg.
@@ db/osqluprec.c @@
+ Please remove debugging statements, and replace fprintf/printf with logmsg.
@@ plugins/newsql/newsql.c @@
+ Please remove debugging statements, and replace fprintf/printf with logmsg.
@@ db/sqlglue.c @@
+ Please make sure that a mutex is unlocked before returning.
d73d901
to
f687669
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Success ✓.
Smoke testing: Error ⚠.
Cbuild submission: Success ✓.
Regression testing: 0/0 tests failed ⚠.
Signed-off-by: Mike Ponomarenko <mponomarenko@bloomberg.net>
f687669
to
7194fb4
Compare
/runtests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ugh failing tests. I have resubmitted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Error ⚠.
Regression testing: 8/524 tests failed ⚠.
The first 10 failing tests are:
sc_repeated_updates_logicalsc_generated [setup failure]
sc_repeated_updates [setup failure]
lock_maxid [setup failure]
selectv_timepart [setup failure]
drop_tblnum [setup failure]
queuedb_rollover
truncatesc
truncatesc_offline_generated
{174259308} Fail db creation if any initial table creation fails