[ 
https://issues.apache.org/jira/browse/ACCUMULO-4317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15339610#comment-15339610
 ] 

Josh Elser commented on ACCUMULO-4317:
--------------------------------------

[~mastermikey...@gmail.com], ok, this is what I'm thinking. We can look at 
applying your patch as-is to 1.6, 1.7 and 1.8. For master, we can look at 
reworking some of these static methods to be able to use some mocking to write 
unit tests which verify your fix.

One question for you though, Michiel: did you verify that this change prevented 
the too many open files error you originally saw if you reverted your change to 
/etc/hosts?

> Accumulo client causes 'too many files open' due to infinite loop.
> ------------------------------------------------------------------
>
>                 Key: ACCUMULO-4317
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4317
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.6.5, 1.7.0, 1.7.1
>            Reporter: Michiel Vanderlee
>            Assignee: Michiel Vanderlee
>            Priority: Blocker
>             Fix For: 1.6.6, 1.7.3, 1.8.1
>
>         Attachments: Accumulo-4317.patch
>
>
> Accumulo stores hostnames in zookeeper, if the client can not resolve these 
> then it will continue to try to connect in a while(true) loop. This will 
> eventually cause 'too many files open' errors.
> Loop is in ServerClient.java$executeRaw
> Bug: Should error out after some time, not retry infintely.
> Workaround: Add hostnames to /etc/hosts and restart.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to