On Tue, May 07, 2019 at 03:41:50PM +0800, John Naylor wrote: > Attached is an attempt to match surrounding code. More broadly, > though, it seems the "ID info" comments belong with the SET_LOCKTAG_* > macros rather than with the LockTagType enum members.
+ LOCKTAG_SPECULATIVE_TOKEN, /* for speculative insertion */ + /* ID info for a speculative token is TRANSACTION info + token */ Shouldn't the first comment be just "speculative insertion"? And the second one "ID info for a speculative insertion is transaction ID + its speculative insert counter"? -- Michael
signature.asc
Description: PGP signature
