Skip to content
This repository has been archived by the owner on Jun 7, 2021. It is now read-only.

Close statement in finally block #1291

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

eheewag
Copy link
Contributor

@eheewag eheewag commented Nov 3, 2017

No description provided.

@Traf-Jenkins
Copy link

Can one of the admins verify this patch?

3 similar comments
@Traf-Jenkins
Copy link

Can one of the admins verify this patch?

@Traf-Jenkins
Copy link

Can one of the admins verify this patch?

@Traf-Jenkins
Copy link

Can one of the admins verify this patch?

@svarnau
Copy link
Contributor

svarnau commented Nov 4, 2017

jenkins, ok to test

@Traf-Jenkins
Copy link

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2164/

@Traf-Jenkins
Copy link

@DaveBirdsall
Copy link
Contributor

What JIRA is associated with this change?

@uamgo
Copy link
Contributor

uamgo commented Nov 23, 2017

Hi Dave, it seems no JIRA for it, but it's helpful for Jekins testing.

@selvaganesang
Copy link
Contributor

Is this PR related to TRAFODION-2704?

@selvaganesang
Copy link
Contributor

If so, can you please update the title of the PR with this JIRA number like [TRAFODION-2704] for automatic linking of the PR to JIRA.

@Traf-Jenkins
Copy link

Can one of the admins verify this patch?

3 similar comments
@Traf-Jenkins
Copy link

Can one of the admins verify this patch?

@Traf-Jenkins
Copy link

Can one of the admins verify this patch?

@Traf-Jenkins
Copy link

Can one of the admins verify this patch?

@Traf-Jenkins
Copy link

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2291/

@Traf-Jenkins
Copy link

@selvaganesang
Copy link
Contributor

Please update the heading of the PR with JIRA number

@Traf-Jenkins
Copy link

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2891/

@Traf-Jenkins
Copy link

@Traf-Jenkins
Copy link

Can one of the admins verify this patch?

1 similar comment
@Traf-Jenkins
Copy link

Can one of the admins verify this patch?

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

Successfully merging this pull request may close these issues.

6 participants