The code in void CHThorFirstNActivity::ready()
gets optimized into
by gcc 8.1 and later, resulting in incorrect behaviour when limit is 0x7fffffffffffffff (i.e. ALL) and skip is positive.
Roxie and thor have similar code constructs but use unsigned __int64 members and therefore do not fail (though there is some iffy code in CFirstNSlaveBase::getMetaInfo which is not allowing for skip when setting info.totalRowsMax