You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@empire-db.apache.org by "Rainer Döbele (JIRA)" <em...@incubator.apache.org> on 2018/10/19 10:00:00 UTC

[jira] [Closed] (EMPIREDB-263) Not possible to set skipRows unless limitRows is also set

     [ https://issues.apache.org/jira/browse/EMPIREDB-263?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rainer Döbele closed EMPIREDB-263.
----------------------------------
    Resolution: Incomplete

No verified implementation received

> Not possible to set skipRows unless limitRows is also set
> ---------------------------------------------------------
>
>                 Key: EMPIREDB-263
>                 URL: https://issues.apache.org/jira/browse/EMPIREDB-263
>             Project: Empire-DB
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: empire-db-2.4.6
>            Reporter: Shaun Forbes
>            Priority: Minor
>
> For databases that support skipRows() and limitRows() when generating a select statement if limitRows hasn't been specified then skipRows isn't checked.
> In Postgres at least it's legal to specify an offset without also specifying a limit, but because Empire-db won't apply the offset unless you also specify a limit it's not possible to generate a select statement with just an offset.
> I understand that the two are generally used together, and I only found this because of a bug in my code, but I don't see a reason why utilizing skipRows() should not be allowed without limitRows(). Unfortunately, it means that the semantics of clearLimit() may need to change since it currently clears both and there's no clearSkip() method.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)