Offset may be specified without a limit. Causes syntax error in mysql, sqlite and maybe others

[YonmaN]

...
$sql = new Sql($adapter);
$select = $sql->select('table');
$select->offset($offset);
$statement = $sql->prepareStatementForSqlObject($select);
$result = $statement->execute();
....

Will throw a syntax error in mysql and sqlite

General error: 1 near "OFFSET": syntax error

[samsonasik]

@YonmaN I created PR #5643 to fix it

[ralphschindler]

I think instead we should just throw an exception for the platforms that requires both to be set. If someone is using this driver to produce valid cross platform abstracted queries, I think it would be better to throw an exception so they can fix their query. Thoughts? (The exception would be thrown in the MysqlDecorator).

[samsonasik]

@ralphschindler imho, I think the exception should not be added because on case that we don't want to limit, it will cause issue, on #5643 I remove cast (int) on $this->limit to make it possible to use maximum of unsigned bigint ( 18446744073709551615 )

[weierophinney]

Fixed with #5940 , to release with 2.3.0.