Sounds about right, though I think we can shave some words off that. How about
int64 maxSpace; /* maximum amount of space occupied among sort * of groups, either in-memory or on-disk */ bool isMaxSpaceDisk; /* true when maxSpace tracks on-disk space, false * means in-memory */ ?