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

Include table visualization in pulses and alerts #7022

Merged
merged 3 commits into from Mar 2, 2018
Merged

Include table visualization in pulses and alerts #7022

merged 3 commits into from Mar 2, 2018

Commits on Mar 2, 2018

  1. Add support for larger pulse/alert tables 

    Previously pulses and alerts that used a table visualization were
limited to 3 columns and 10 rows. This commit changes the backed to
bump that to 10 columns and 20 rows. When a resultset has more data
than will fit into a 10x20 table, it will trigger a CSV attachment
that will include the full results of the query, up to 2000 results.

This commit also limits all queries to 2000 rows, similar to how queries
issued via the UI are constrained. Users wanting more than 2000 rows
use the existing export functionality.
    @iethree
    iethree committed Mar 2, 2018
    Configuration menu
    Copy the full SHA
    9735916 View commit details
    Browse the repository at this point in the history

  2. Frontend changes to support larger pulse/alert tables

    @tlrobinson @iethree
    tlrobinson authored and iethree committed Mar 2, 2018
    Configuration menu
    Copy the full SHA
    b1450b2 View commit details
    Browse the repository at this point in the history

  3. Style changes for larger pulse/alert tables

    @mazameli @iethree
    mazameli authored and iethree committed Mar 2, 2018
    Configuration menu
    Copy the full SHA
    de4343a View commit details
    Browse the repository at this point in the history