Skip to content
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

RDBBtree.SindexEraseRange unit test failing #2042

Closed
srh opened this issue Feb 28, 2014 · 11 comments
Closed

RDBBtree.SindexEraseRange unit test failing #2042

srh opened this issue Feb 28, 2014 · 11 comments
Milestone

Comments

@srh
Copy link
Contributor

srh commented Feb 28, 2014

[ RUN      ] RDBBtree.SindexEraseRange
src/unittest/rdb_btree.cc:309: Failure
Value of: groups->size()
  Actual: 0
Expected: 1
src/unittest/rdb_btree.cc:309: Failure
Value of: groups->size()
  Actual: 0
Expected: 1
src/unittest/rdb_btree.cc:309: Failure
Value of: groups->size()
  Actual: 0
Expected: 1
src/unittest/rdb_btree.cc:309: Failure
Value of: groups->size()
  Actual: 0
Expected: 1
src/unittest/rdb_btree.cc:309: Failure
Value of: groups->size()
  Actual: 0
Expected: 1
[  FAILED  ] RDBBtree.SindexEraseRange (33461 ms)

I get this in next as of d951936.

@srh srh added this to the 1.12 milestone Feb 28, 2014
@srh
Copy link
Contributor Author

srh commented Feb 28, 2014

This test passes in d3205c7 and fails in eea0c65, so it seems like it was not passing in the 1096_gmr branch when that was merged into next.

@mlucy mlucy self-assigned this Feb 28, 2014
@mlucy
Copy link
Member

mlucy commented Feb 28, 2014

Assigning to myself.

@AtnNn AtnNn assigned AtnNn and unassigned mlucy Mar 1, 2014
@AtnNn AtnNn closed this as completed in a48c3f0 Mar 3, 2014
@anatol
Copy link
Contributor

anatol commented Jun 17, 2014

I see similar test failure with 1.13.0 release on Linux Arch

[----------] 4 tests from RDBBtree
[ RUN      ] RDBBtree.SindexPostConstruct
[       OK ] RDBBtree.SindexPostConstruct (772 ms)
[ RUN      ] RDBBtree.SindexEraseRange
src/unittest/rdb_btree.cc:312: Failure
Value of: groups->size()
  Actual: 1
Expected: 0
src/unittest/rdb_btree.cc:312: Failure
Value of: groups->size()
  Actual: 1
Expected: 0
src/unittest/rdb_btree.cc:312: Failure
Value of: groups->size()
  Actual: 1
Expected: 0
src/unittest/rdb_btree.cc:312: Failure
Value of: groups->size()
  Actual: 1
Expected: 0

@mlucy mlucy modified the milestones: 1.13.x, 1.12 Jun 17, 2014
@mlucy
Copy link
Member

mlucy commented Jun 17, 2014

Moving to 1.13.x. Thanks for the report @anatol .

@coffeemug
Copy link
Contributor

Reopening.

@coffeemug coffeemug reopened this Jun 17, 2014
@AtnNn AtnNn removed their assignment Jul 12, 2014
@coffeemug coffeemug modified the milestones: 1.13.x, 1.14.x Aug 14, 2014
@coffeemug coffeemug modified the milestones: 1.14.x, 1.15.x Sep 29, 2014
@anatol
Copy link
Contributor

anatol commented Nov 12, 2014

Happened to me when I was building rethinkdb 1.15.2

[----------] 4 tests from RDBBtree
[ RUN      ] RDBBtree.SindexPostConstruct
[       OK ] RDBBtree.SindexPostConstruct (991 ms)
[ RUN      ] RDBBtree.SindexEraseRange
src/unittest/rdb_btree.cc:321: Failure
Value of: groups->size()
  Actual: 1
Expected: 0

@srh
Copy link
Contributor Author

srh commented Nov 12, 2014

I'll take a look.

@srh srh self-assigned this Nov 12, 2014
@srh
Copy link
Contributor Author

srh commented Nov 12, 2014

Running 1000 iterations of the test revealed nothing, they all passed. I'm going to change the test to actually reveal what the contents of groups is when its size is non-zero, because that would help the next time we see it.

@srh
Copy link
Contributor Author

srh commented Nov 12, 2014

Review 2301, branch sam_2042 (based off v1.15.x) has the error message improvement.

@srh
Copy link
Contributor Author

srh commented Nov 14, 2014

The unittest error message changes (and a bunch of debug_print functions) are in v1.15.x as of 710cecdcN8d65d6735c43f1df007a01d.

@srh srh removed their assignment Nov 14, 2014
@danielmewes danielmewes modified the milestones: 1.15.x, 1.16.x Jan 29, 2015
@danielmewes
Copy link
Member

This has probably been fixed by the fix for #3805.

@AtnNn AtnNn modified the milestones: 1.16.x, 2.0 Mar 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants