Limit 0 or null as parameter should return all records

[dszymczuk]

Example (I use rethinkdbdash):

r.table(tableNotification).orderBy(r.desc('time')).limit(limit)
is better than:

if(limit > 0){
    r.table(tableNotification).orderBy(r.desc('time')).limit(limit).run()
} else {
    r.table(tableNotification).orderBy(r.desc('time')).run()
}

After comment @danielmewes in rethinkdb/docs#1133 I think, that null will be good to get all records.
Also nth and skip should the same option.

[danielmewes]

Thanks for re-opening the issue here.

I'm marking this as a ReQL_proposal. We'll discuss it for one of the upcoming releases.

[VeXocide]

I agree with @danielmewes that we shouldn't special case 0 and have limit(0) return zero documents, that's what you've asked for after all. Adding support for limit(null) on the other hand sounds reasonable.

Not sure what nth(null) would do though, and I'm not sure there's much use for a skip(null) that skips all documents either.

[neonrust]

I think, at least limit(null) and skip(null) have intuitive English meanings; "no limit" and "skip nothing"... (and as usual :) even more intuitive in Python: limit(None) and skip(None))
However, nth(null) is less intuitive... if it even makes any sense?

[dszymczuk]

I would like to focus on limit. Because when I want to use limit, I have to use construction with "if" as above.
It's not comfortable :)

You're right - nth doesn't make sense.