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.

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

[Manual Backport] Allow arbitrary report record counts #428

Merged
merged 2 commits into from
Sep 9, 2024

Commits on Sep 9, 2024

  1. [Feature] Add the ability to configure record limit/count from the UI (

    …opensearch-project#395)
    
    * feat(components/report-settings): add field for variable record limits
    
    Signed-off-by: Károly Szakály <[email protected]>
    
    * fix(components/report-settings): set type for event target
    
    Signed-off-by: Károly Szakály <[email protected]>
    
    * feat(components/report-settings): load existing record limit in edit mode
    
    Signed-off-by: Károly Szakály <[email protected]>
    
    * feat(components/report-definition-details): show record limit on report definition details page
    
    Signed-off-by: Károly Szakály <[email protected]>
    
    * feat(components/report-definition-details): omit record limit for non saved search sources
    
    Signed-off-by: Károly Szakály <[email protected]>
    
    * feat(components/report-details): show record limit on report details page
    
    Signed-off-by: Károly Szakály <[email protected]>
    
    * tests(cypress): add `force: true` to name input
    
    Signed-off-by: Károly Szakály <[email protected]>
    
    * tests(jest/snapshots): update snapshots
    
    Signed-off-by: Károly Szakály <[email protected]>
    
    * feat(components/report-settings): add proper i18n label for record limit input
    
    Signed-off-by: Károly Szakály <[email protected]>
    
    * Update release notes
    
    Signed-off-by: Simeon Widdis <[email protected]>
    
    * Add potential OOM warning for large record limits
    
    Signed-off-by: Simeon Widdis <[email protected]>
    
    * Make memory warning more conservative based on original limit
    
    Signed-off-by: Simeon Widdis <[email protected]>
    
    ---------
    
    Signed-off-by: Károly Szakály <[email protected]>
    Signed-off-by: Simeon Widdis <[email protected]>
    Co-authored-by: Simeon Widdis <[email protected]>
    Signed-off-by: Simeon Widdis <[email protected]>
    szkly and Swiddis committed Sep 9, 2024
    Configuration menu
    Copy the full SHA
    76d38d8 View commit details
    Browse the repository at this point in the history
  2. Fix merge conflict markers

    Signed-off-by: Simeon Widdis <[email protected]>
    Swiddis committed Sep 9, 2024
    Configuration menu
    Copy the full SHA
    e67c06a View commit details
    Browse the repository at this point in the history