Skip to content

Convert delete expunge to use Spring Batch #2697

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
Merged
fil512 merged 77 commits into from
Jun 15, 2021
Merged

Convert delete expunge to use Spring Batch #2697

fil512 merged 77 commits into from
Jun 15, 2021

Conversation

@fil512
Copy link
Collaborator

@fil512 fil512 commented Jun 1, 2021

No description provided.

fil512 added 30 commits May 24, 2021 09:53
@fil512
prepare to add $delete-expunge operation that will create a spring ba…
a90899d 
…tch job
@fil512
Merge remote-tracking branch 'origin/master' into ks-20210523-batch-d…
60ae5cd 
…elete-expunge
@fil512
Add operation
cfd0ac6
@fil512
Wire up jpa provider. Begin with failing test.
3aa0f39
@fil512
Copy/paste bulk import job as a starting point.
5480d9e 
FIXME with proposed design
@fil512
Merge remote-tracking branch 'origin/master' into ks-20210523-batch-d…
1755264 
…elete-expunge
@fil512
delete expunge job parameter validation with test
43236fd
@fil512
Merge remote-tracking branch 'origin/master' into ks-20210523-batch-d…
a73eec4 
…elete-expunge
@fil512
implemented reader
5f92b43 
stubbed processor, writer
@fil512
wip for master merge
edecfc5
@fil512
Merge remote-tracking branch 'origin/master' into ks-20210523-batch-d…
ff8526e 
…elete-expunge
@fil512
Merge remote-tracking branch 'origin/master' into ks-20210523-batch-d…
3e14155 
…elete-expunge
@fil512
started implementing reader
9faa7ad
@fil512
started implementing reader
5699ee4
@fil512
working with stubs
bc7fbaa
@fil512
Merge remote-tracking branch 'origin/master' into ks-20210523-batch-d…
bf7d462 
…elete-expunge
@fil512
merged master resolved conflicts
e37cbdc
@fil512
happy path batch delete expunge is done
efaf02e
@fil512
Provider done but test not passing. Guessing batch infrastructure not…
44bf616 
… running in that test.
@fil512
IT test works now
f99d7a1
@fil512
Merge remote-tracking branch 'origin/master' into ks-20210523-batch-d…
21c39d0 
…elete-expunge
@fil512
add reader test
2e76bdf
@fil512
Converted delete _expunge=true to use new batch job
bf3e6d6
@fil512
DeleteExpungeDaoTest passes
8b349e5
@fil512
Fix test
6af0e79
@fil512
Change batch size to integer
6289327
@fil512
rename search count to batch size
91dd42e
@fil512
Merge remote-tracking branch 'origin/master' into ks-20210523-batch-d…
687f143 
…elete-expunge
@fil512
Make delete expunge partition aware
ce165f7
@fil512
updated docs
cc9da0f
@lgtm-com
Copy link

lgtm-com bot commented Jun 4, 2021

This pull request introduces 1 alert when merging 876aa8a into 0143e83 - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jun 4, 2021

This pull request introduces 1 alert when merging 817717b into 0143e83 - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jun 5, 2021

This pull request introduces 1 alert when merging 4b5d2c6 into 4f1f09a - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jun 7, 2021

This pull request introduces 1 alert when merging bb65822 into dc627dc - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jun 9, 2021

This pull request introduces 1 alert when merging 2cbf5b2 into 3863e82 - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jun 13, 2021

This pull request introduces 1 alert when merging 8fa6875 into 090dc9a - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jun 14, 2021

This pull request introduces 1 alert when merging eb32bfa into 3ee05b5 - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jun 14, 2021

This pull request introduces 1 alert when merging 39f8f8c into 376a84d - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jun 15, 2021

This pull request introduces 1 alert when merging 8e28eb9 into 376a84d - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@lgtm-com
Copy link

lgtm-com bot commented Jun 15, 2021

This pull request introduces 1 alert when merging efa1588 into 376a84d - view on LGTM.com

new alerts:

  • 1 for Spurious Javadoc @param tags

@fil512 fil512 merged commit 134631f into master Jun 15, 2021
@fil512 fil512 deleted the ks-20210523-batch-delete-expunge branch June 15, 2021 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tadgh tadgh tadgh approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@fil512 @tadgh