Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Trim the number of columns we output to stdout #294

Closed
nehanene15 opened this issue Aug 3, 2021 · 0 comments
Closed

Trim the number of columns we output to stdout #294

nehanene15 opened this issue Aug 3, 2021 · 0 comments
Assignees
@yogeshtewari @nehanene15
Labels
priority: p1 High priority. Fix may be included in the next release. Release Next release candidate type: feature request 'Nice-to-have' improvement, new feature or different behavior or design.

Comments

@nehanene15
Copy link
Collaborator

nehanene15 commented Aug 3, 2021 •
edited

Many of the columns that we output for the default stdout result handler are unnecessary and create clutter. This will only apply to the --format table output when we pretty print. For all other formats (text,json,csv) we will output the whole result set.

We can remove the following:

  • Run ID
  • Start time
  • End time
  • Aggregation type (since it is included in the validation name)
  • source_agg
  • target_agg
  • difference
  • pct_threshold
  • labels
@nehanene15 nehanene15 self-assigned this Aug 3, 2021
@nehanene15 nehanene15 added priority: p1 High priority. Fix may be included in the next release. Release Next release candidate type: feature request 'Nice-to-have' improvement, new feature or different behavior or design. labels Aug 3, 2021
@yogeshtewari yogeshtewari self-assigned this Aug 12, 2021
yogeshtewari pushed a commit that referenced this issue Aug 20, 2021
feat: Trim the number of columns we output to stdout (#294)
8567ba1
yogeshtewari pushed a commit that referenced this issue Aug 20, 2021
feat: Trim the number of columns we output to stdout (#294)
f6a0c35
yogeshtewari pushed a commit that referenced this issue Aug 20, 2021
feat: Trim the number of columns we output to stdout (#294)
5e43fcf
yogeshtewari added a commit that referenced this issue Aug 25, 2021
@yogeshtewari
feat: Allow user to specify a format for stdout T2 (#242) (#296)
ec1af22 
* feat: Allow user to specify a format for stdout(#242)

* feat: Allow user to specify a format for stdout. Test case fixes(#242)

* feat: Allow user to specify a format for stdout. Test case fixes(#242)

* feat: Allow user to specify a format for stdout. Default format and Text result handler test case fixes(#242)

* feat: Trim the number of columns we output to stdout (#294)

* feat: Trim the number of columns we output to stdout (#294)

* feat: Trim the number of columns we output to stdout (#294)

Co-authored-by: Yogesh Tewari <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yogeshtewari yogeshtewari

@nehanene15 nehanene15

Labels
priority: p1 High priority. Fix may be included in the next release. Release Next release candidate type: feature request 'Nice-to-have' improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@yogeshtewari @ryanmcdowell @nehanene15