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 <karoly.szakaly2000@gmail.com>
    
    * fix(components/report-settings): set type for event target
    
    Signed-off-by: Károly Szakály <karoly.szakaly2000@gmail.com>
    
    * feat(components/report-settings): load existing record limit in edit mode
    
    Signed-off-by: Károly Szakály <karoly.szakaly2000@gmail.com>
    
    * feat(components/report-definition-details): show record limit on report definition details page
    
    Signed-off-by: Károly Szakály <karoly.szakaly2000@gmail.com>
    
    * feat(components/report-definition-details): omit record limit for non saved search sources
    
    Signed-off-by: Károly Szakály <karoly.szakaly2000@gmail.com>
    
    * feat(components/report-details): show record limit on report details page
    
    Signed-off-by: Károly Szakály <karoly.szakaly2000@gmail.com>
    
    * tests(cypress): add `force: true` to name input
    
    Signed-off-by: Károly Szakály <karoly.szakaly2000@gmail.com>
    
    * tests(jest/snapshots): update snapshots
    
    Signed-off-by: Károly Szakály <karoly.szakaly2000@gmail.com>
    
    * feat(components/report-settings): add proper i18n label for record limit input
    
    Signed-off-by: Károly Szakály <karoly.szakaly2000@gmail.com>
    
    * Update release notes
    
    Signed-off-by: Simeon Widdis <sawiddis@amazon.com>
    
    * Add potential OOM warning for large record limits
    
    Signed-off-by: Simeon Widdis <sawiddis@amazon.com>
    
    * Make memory warning more conservative based on original limit
    
    Signed-off-by: Simeon Widdis <sawiddis@amazon.com>
    
    ---------
    
    Signed-off-by: Károly Szakály <karoly.szakaly2000@gmail.com>
    Signed-off-by: Simeon Widdis <sawiddis@amazon.com>
    Co-authored-by: Simeon Widdis <sawiddis@amazon.com>
    Signed-off-by: Simeon Widdis <sawiddis@amazon.com>
    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 <sawiddis@amazon.com>
    Swiddis committed Sep 9, 2024
    Configuration menu
    Copy the full SHA
    e67c06a View commit details
    Browse the repository at this point in the history