Skip to content

AbstractKiteProcessor.getSchema leaks file handles#101

Closed
jackowaya wants to merge 1 commit intoapache:masterfrom
jackowaya:schemaLeak
Closed

AbstractKiteProcessor.getSchema leaks file handles#101
jackowaya wants to merge 1 commit intoapache:masterfrom
jackowaya:schemaLeak

Conversation

@jackowaya
Copy link
Copy Markdown

Close file handles when reading schema from resources or paths in AbstractKiteProcessor.getSchema

@joey
Copy link
Copy Markdown
Contributor

joey commented Oct 13, 2015

+1, lgtm

@joey
Copy link
Copy Markdown
Contributor

joey commented Oct 13, 2015

Is there a JIRA associated with this?

@jackowaya
Copy link
Copy Markdown
Author

Yes. I screwed up making the commit message. It's https://issues.apache.org/jira/browse/NIFI-1035

Close file handles when reading schema from resources or paths in AbstractKiteProcessor.getSchema
@jackowaya
Copy link
Copy Markdown
Author

Just force pushed with a message that has the jira ID in it. Hopefully that doesn't screw anything up.

@joey
Copy link
Copy Markdown
Contributor

joey commented Oct 13, 2015

No worries, it still looks good to me :)

@rdblue
Copy link
Copy Markdown
Contributor

rdblue commented Oct 13, 2015

+1, thanks for fixing this @jackowaya!

@joewitt
Copy link
Copy Markdown
Contributor

joewitt commented Oct 18, 2015

looks good - am merging. Sorry for the delay Alan

@asfgit asfgit closed this in 943ccfe Oct 18, 2015
JPercivall pushed a commit to JPercivall/nifi that referenced this pull request Apr 23, 2018
…and making use of a mirror instead of Apache dist.

Signed-off-by: joewitt <joewitt@apache.org>
iadamcsik pushed a commit to iadamcsik/nifi that referenced this pull request Oct 22, 2025
- CDPDFX-6813 Fix use of flow comparator in standard flow change event handler
- CDPDFX-6813 Remove duplicate NAR in assembly
- CDPDFX-6831 Fix publishing of controller service bulletins (apache#101)
- CDPDFX-6854: Correcting process group ID mapping in flow bulletin task (apache#102)
- CDPDFX-6848: Ensuring that all child components are included in each published BulletinGroup. This separation allows them to not be pruned when there are many other descendent components.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants