Order by desc is broken. Tests such as Types_15, Types_17 fail.
Patch committed to trunk. Thanks Rohini for review!
Good point. We shall consistently use the successor for the properties. Update the patch.
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.