-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add routine to disconnect all invisible children of subjects. #279
Conversation
I test tok det litt under to minutter å kjøre publisering med dette. Merk at ingen noder faktisk slettes, men kun frakobles fra foreldrenoden. |
bf78d78
to
bd8b7fa
Compare
Not 22
bd8b7fa
to
bfd8c21
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, bare et spm!
deleter.setVersion(schemaFromHash(hash)); | ||
executor.submit(deleter); | ||
} catch (Exception e) { | ||
logger.info(e.getMessage(), e); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jeg husker vi snakka om det, men jeg husker ikke helt hva det var igjen. Hvorfor burde ikke denne være logger.error
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Det kunne den fint vore, men tenkte berre at det ikkje gav så masse ekstra verdi. Vil sannsynligvis ikkje skje uansett.
Lokalt går dette gjennom på noen minutter, men vil teste å lage en release som eg kjører på test.