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.

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

[JBPM-7377] LogCleanupCommand deletes completed sub process logs whil… #1258

Closed
wants to merge 1 commit into from
Closed

[JBPM-7377] LogCleanupCommand deletes completed sub process logs whil… #1258

wants to merge 1 commit into from

Conversation

tkobayas
Copy link
Contributor

…e its parent process is alive

  • Unit test

@mswiderski
Copy link
Contributor

@mareknovotny this is test only and I am not really convinced it should be implemented see my comment in Jira

@mareknovotny
Copy link
Member

@mswiderski yep, I get it. I will follow up on JIRAs.

@mareknovotny mareknovotny added for-CP-waiting do not merge, this is just in queue for backporting payload DO_NOT_MERGE labels Jul 4, 2018
@kie-qe-ci
Copy link

Can one of the admins verify this patch?

@mswiderski
Copy link
Contributor

corresponding jiras were closed os closing this PR too

@mswiderski mswiderski closed this Aug 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mswiderski mswiderski Awaiting requested review from mswiderski

@MarianMacik MarianMacik Awaiting requested review from MarianMacik

Assignees
No one assigned
Labels
DO_NOT_MERGE for-CP-waiting do not merge, this is just in queue for backporting payload
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tkobayas @mswiderski @mareknovotny @kie-qe-ci