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

Time filter optimize approach #1

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

shriya-deshmukh
Copy link
Owner

Problem Statement

  • Problem statement

Design

  • For Bug describe the fix here.
  • For Feature, Post the link to the solution page on the confluence CORTX Foundation Library

Coding

  • Coding conventions are followed and code is consistent [Y/N]:
  • Confirm All CODACY errors are resolved [Y/N]:

Testing

  • Are test cases updated along with code changes due to Enhancements/Bugs [Y/N]:
  • Confirm that new test cases are added to regression and sanity plan files and relevant feature plan files [Y/N]:
  • Confirm that Test Cases are added for new features added [Y/N]:
  • Confirm Test Cases cover Happy Path, Non-Happy Path and Scalability [Y/N]:
  • Confirm Testing was performed with installed RPM/K8s deployment [Y/N]:

Review Checklist

Before posting the PR please ensure:

  • PR is self reviewed
  • Is there a change in filename/package/module or signature [Y/N]:
  • If yes for above point, Is a notification sent to all other cortx components [Y/N]
  • New package/s added to setup.py?
  • Jira is updated
  • Check if the description is clear and explained.
  • Check Acceptance Criterion is defined.
  • All the tests performed should be mentioned before Resolving a JIRA.
  • Verification needs to be done before marked as Closed/Verified.

KVstore/Confstore feature (changes/fixes) checklist

Confirm changes are made for:

  • ConfStore
  • KVStore
  • ConfCli
  • Test cases added for all above 3
  • changes done in all the KVpayloads (ConsulKvPayload, IniKvPayload, KvPayload)

Documentation

  • Changes done to WIKI / Confluence page

…pod logs. (Seagate#875)

Signed-off-by: Shriya Deshmukh <shriya.deshmukh@seagate.com>
* CORTX-34050: Apply limit_time filter on .gz files.

Signed-off-by: Shriya Deshmukh <shriya.deshmukh@seagate.com>
Signed-off-by: Shriya Deshmukh <shriya.deshmukh@seagate.com>
Signed-off-by: Shriya Deshmukh <shriya.deshmukh@seagate.com>
Signed-off-by: Shriya Deshmukh <shriya.deshmukh@seagate.com>
@shriya-deshmukh
Copy link
Owner Author

shriya-deshmukh commented Sep 12, 2022

Optimize approach for time filter - Seagate#877 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant