steveloughran opened a new pull request, #5186:
URL: https://github.com/apache/hadoop/pull/5186

   
   ### Description of PR
   
   removes a needless line in `AvroFSInput(final FileContext fc, final Path p` 
which opens the path a second time and discards the result.
   
   on input streams which immediately set up a connection with the store, this 
can leak connections.
   
   Note, this is through the FileContext constructor, which is only used in 
production in org.apache.hadoop.fs.shell.Display, which being used from the 
command line, shouldn't be a long-lived process
   
   ### How was this patch tested?
   
   going to see what yetus says.
   
   ### For code changes:
   
   - [X] Does the title or this PR starts with the corresponding JIRA issue id 
(e.g. 'HADOOP-17799. Your PR title ...')?
   - [ ] Object storage: have the integration tests been executed and the 
endpoint declared according to the connector-specific documentation?
   - [ ] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?
   - [ ] If applicable, have you updated the `LICENSE`, `LICENSE-binary`, 
`NOTICE-binary` files?
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to