You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2022/05/20 21:48:04 UTC

[GitHub] [ozone] smengcl opened a new pull request, #3440: HDDS-6784. Missing close() for two RocksDB table iterators

smengcl opened a new pull request, #3440:
URL: https://github.com/apache/ozone/pull/3440

   ## What changes were proposed in this pull request?
   
   There are two places where the RocksDB table iterators aren't properly closed. Inspired by https://github.com/apache/ozone/pull/3131#discussion_r878296430
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-6784
   
   ## How was this patch tested?
   
   - All existing tests should pass.


-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


[GitHub] [ozone] adoroszlai commented on pull request #3440: HDDS-6784. Missing close() for two RocksDB table iterators

Posted by GitBox <gi...@apache.org>.
adoroszlai commented on PR #3440:
URL: https://github.com/apache/ozone/pull/3440#issuecomment-1133951492

   Thanks @smengcl for the patch, @kaijchen for the review.


-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


[GitHub] [ozone] adoroszlai merged pull request #3440: HDDS-6784. Missing close() for two RocksDB table iterators

Posted by GitBox <gi...@apache.org>.
adoroszlai merged PR #3440:
URL: https://github.com/apache/ozone/pull/3440


-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


[GitHub] [ozone] smengcl commented on pull request #3440: HDDS-6784. Missing close() for two RocksDB table iterators

Posted by GitBox <gi...@apache.org>.
smengcl commented on PR #3440:
URL: https://github.com/apache/ozone/pull/3440#issuecomment-1134937646

   Thanks @kaijchen for the review. Thanks @adoroszlai for merging this.


-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org