[ 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.