[SERVER-38494] EphemeralForBtreeImplTestHarnessHelper builds SortedDataInterface with empty index key specification Created: 09/Dec/18 Updated: 27/Oct/23 Resolved: 22/Jul/20 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | 4.1.6 |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | James Wahlin | Assignee: | Backlog - Storage Execution Team |
| Resolution: | Gone away | Votes: | 0 |
| Labels: | intern_ephemeral_for_test | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Assigned Teams: |
Storage Execution
|
||||||||||||
| Participants: | |||||||||||||
| Description |
|
The EphemeralForBtreeImplTestHarnessHelper class builds a SortedDataInterface with an empty index key specification. This leads the BuilderAddSameKey test in the storage_ephemeral_for_test_btree_test suite, which generates a DuplicateKey exception, to call buildDupKeyErrorStatus in index_entry_comparison.cpp with an empty key definition and a non-empty set of key values. This behavior prevents us from adding stronger invariants to buildDupKeyErrorStatus, such as the following:
This would be a good addition as this method iterates over both key and values in a single loop, with the assumption that they contain the same number of elements. |
| Comments |
| Comment by Eric Milkie [ 14/Jan/19 ] |
|
We can resolve this when ephemeralForTest is deleted. |