[ 
https://issues.apache.org/jira/browse/CASSANDRA-1178?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12877305#action_12877305
 ] 

Masood Mortazavi edited comment on CASSANDRA-1178 at 6/9/10 11:53 PM:
----------------------------------------------------------------------

Sounds reasonable . . . 

It seems to me that he idea is that 

(1) If you pass a non-null file object reference to the constructor, you are 
the best judge of when to close it. 

(2) If you pass a null, instead, you will have to ask SSTableSliceIterator to 
clean up after you're done by calling "close" on it. 

This is then the semantics of this class. 

I think it may be useful to include this contractual fact in the javadoc for 
this class. 

(Side note: It would probably be best not to allow IDE's to change the order of 
imports from the one that's common everywhere else unless there is a policy by 
this project to make such reordering. It is better to keep things consistent.)



      was (Author: mortazavi):
    Sounds reasonable . . . 

It seems to me that he idea is that 

(1) If you pass a non-null file object reference to the constructor, you are 
the best judge of when to close it. 

(2) If you pass a null, instead, you will have to ask SSTableSlideIterator to 
clean up after you're done by calling "close" on it. 

This is then the semantics of this class. 

I think it may be useful to include this contractual fact in the javadoc for 
this class. 

(Side note: It would probably be best not to allow IDE's to change the order of 
imports from the one that's common everywhere else unless there is a policy by 
this project to make such reordering. It is better to keep things consistent.)
  
> get_slice calls do not close files when finished resulting in "too many open 
> files" exceptions and rendering C unusable
> -----------------------------------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-1178
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-1178
>             Project: Cassandra
>          Issue Type: Bug
>    Affects Versions: 0.7
>            Reporter: Matthew F. Dennis
>            Assignee: Matthew F. Dennis
>             Fix For: 0.7
>
>         Attachments: 0001-trunk-1178.patch
>
>
> insert ~100K rows.  Read them back in a loop.  Notice "too many open files" 
> exceptions in log.  SSTableSliceIterator is never closing the files.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to