get runs CLI command doesn't limit number of records queried by default

Description

If user does not specify a limit, the CLI default to Int.MAX_VALUE:

https://github.com/caskdata/cdap/blob/release/5.0/cdap-cli/src/main/java/co/cask/cdap/cli/command/GetProgramRunsCommand.java#L59

Instead, it should not pass the limit parameter to the backend and perhaps left the backend server default to whatever value it defaults to on its own.

Two other places we do this are for getting stream events:
https://github.com/caskdata/cdap/blob/develop/cdap-cli/src/main/java/co/cask/cdap/cli/command/GetStreamEventsCommand.java#L57

and get program logs:

https://github.com/caskdata/cdap/blob/release/5.0/cdap-cli/src/main/java/co/cask/cdap/cli/command/GetProgramLogsCommand.java#L53

Release Notes

None

Activity

Show:
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Components

Priority

Created August 29, 2018 at 10:38 PM
Updated September 20, 2018 at 12:27 AM