get runs CLI command doesn't limit number of records queried by default
Description
Release Notes
None
Activity
Show:
Pinned fields
Click on the next to a field label to start pinning.
Details
Details
Assignee

Reporter

Components
Priority
Created August 29, 2018 at 10:38 PM
Updated September 20, 2018 at 12:27 AM
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