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

Run eventual scans in ComprehensiveIT #4705

Open
keith-turner opened this issue Jun 26, 2024 · 1 comment · May be fixed by #4754
Open

Run eventual scans in ComprehensiveIT #4705

keith-turner opened this issue Jun 26, 2024 · 1 comment · May be fixed by #4754
Assignees
Labels
enhancement This issue describes a new feature, improvement, or optimization.
Milestone

Comments

@keith-turner
Copy link
Contributor

Is your feature request related to a problem? Please describe.

ComprehensiveIT tries to touch as much of the Accumulo public API as possible in a singe test. The test does not currently run eventual scans. Would be good to add this to ComprehensiveIT to know if the feature is working at the most basic level.

Describe the solution you'd like

Add a new test or modify an existing one in ComprehensiveIT to run eventual scans. Should be done in such a way that its easy to verify that eventual scans are running, can do something like the following.

  1. Add data to table
  2. Run eventual scan
  3. Add new data to table
  4. Run 2nd eventual scan, should not see new data.
@keith-turner keith-turner added the enhancement This issue describes a new feature, improvement, or optimization. label Jun 26, 2024
@ArbaazKhan1
Copy link
Contributor

I can take a look at this

@ctubbsii ctubbsii added this to the 3.1.0 milestone Jul 12, 2024
@ArbaazKhan1 ArbaazKhan1 linked a pull request Jul 24, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This issue describes a new feature, improvement, or optimization.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants