-
Type: Improvement
-
Resolution: Done
-
Priority: Major - P3
-
None
-
Affects Version/s: None
-
Component/s: None
-
Labels:None
There are a number of places in our code now that have comments using the abbreviation hs - that is the abbreviation we use within variable names, but comments should be written as completed English sentences, so should use history store.
My quick search showed up the following places:
% ack "\*.* hs " src src/cursor/cur_hs.c 150: * WT_CURSOR->next method for the hs cursor type. 181: * WT_CURSOR->prev method for the hs cursor type. 212: * WT_CURSOR->close method for the hs cursor type. 266: * WT_CURSOR->set_key method for the hs cursor type. 484: * WT_CURSOR->search_near method for the hs cursor type. 594: * WT_CURSOR->get_key method for the hs cursor type. 617: * WT_CURSOR->get_value method for the hs cursor type. 640: * WT_CURSOR->set_value method for the hs cursor type. 661: * WT_CURSOR->insert method for the hs cursor type. 729: * WT_CURSOR->remove method for the hs cursor type. 781: * WT_CURSOR->update method for the hs cursor type. src/include/wiredtiger.in 5841: * transaction: rollback to stable hs records with stop timestamps older 6445: * transaction: rollback to stable hs records with stop timestamps older src/history/hs_rec.c 1058: * Increment the hs counter that we'll be using to insert with