[SERVER-58790] RLE in Simple8b fails when using 0 as added Created: 23/Jul/21  Updated: 29/Oct/23  Resolved: 10/Aug/21

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: 5.1.0
Fix Version/s: 5.1.0-rc0

Type: Bug Priority: Major - P3
Reporter: Luke Bhan (Inactive) Assignee: Jackson Xie (Inactive)
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Backwards Compatibility: Fully Compatible
Operating System: ALL
Steps To Reproduce:

Simple8b s8b;
763
764 std::vector<uint64_t> expectedInts(1920, 0);
765 for(uint32_t i = 0; i < 200; ++i)

{ 766 expectedInts.push_back(2); 767 }

768 testAppendAndGetAllInts(s8b, expectedInts);

Sprint: Execution Team 2021-08-23
Participants:

 Description   

This will fail to even pass just testAppendAndGetAllInts. Im pretty sure it applies to any situation in which we exactly fill up the rle buffer with 0s and then try to add another value that will also use rle. Pretty sure its because we assume 0 as a base but then if the user also adds 0 we get doubled. A simple bugfix would be to never start rle if we haven't at least written one s8b word to buffer. May not be best solution though!



 Comments   
Comment by Vivian Ge (Inactive) [ 06/Oct/21 ]

Updating the fixversion since branching activities occurred yesterday. This ticket will be in rc0 when it’s been triggered. For more active release information, please keep an eye on #server-release. Thank you!

Comment by Githook User [ 10/Aug/21 ]

Author:

{'name': 'Jackson Xie', 'email': 'jackson.xie@mongodb.com', 'username': 'jacksonx9'}

Message: SERVER-58790: add Simple8b test for RLE 0 then immediately switching to another value for RLE
Branch: master
https://github.com/mongodb/mongo/commit/3888d215464a081c831074918ad590a8e3ba74a1

Comment by Henrik Edin [ 02/Aug/21 ]

Check if this is still an issue with our recent changes.

Generated at Thu Feb 08 05:45:27 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.