influxdata / influxdata/influxdb-java

Should contain 0

Open
#959 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Java
Stars
1.2k
Forks
469
PR merge metrics
No merged PRs in 30d

Description

public Select limit(int limit, long offSet) {
if (limit > 0 && offSet > 0L) {
this.limit = Optional.of(limit);
this.offSet = Optional.of(offSet);
return this;
} else {
throw new IllegalArgumentException("Invalid LIMIT and OFFSET Value, must be strictly positive");
}
}

influxdb allow ” limit = 0 and offset =0 “

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the Select.limit(int limit, long offSet) implementation shown in the issue and inspect nearby tests, if present. Update the validation behavior so zero values are accepted while negative values remain invalid, then run the relevant test suite or add coverage if existing tests do not cover the boundary cases.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api, databases
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.