Hi
In the function heapgetpage. If a table is not updated very frequently.
Many actions in tuple loops are superfluous. For all_visible pages,
loctup does not need to be assigned, nor does the "valid" variable.
CheckForSerializableConflictOutNeeded from
HeapCheckForSerializableConflictOut function, it only need to inspect at
the beginning of the cycle only once. Using vtune you can clearly see
the result (attached heapgetpage.jpg).
So by splitting the loop logic into two parts, the vtune results show
significant improvement (attached heapgetpage-allvis.jpg).
The test data uses TPC-H's table "orders" with a scale=20, 30 million rows.
Quan Zongliang
