Order by desc is broken. Tests such as Types_15, Types_17 fail.
Is there a purpose to storing the sort order in the predecessor operator, but the global sort property in the successor? Other than that this looks good.
Good point. We shall consistently use the successor for the properties. Update the patch.
Patch committed to trunk. Thanks Rohini for review!