{"comment":{"url":"https://api.github.com/repos/LLNL/zfs/comments/17564390","html_url":"https://github.com/LLNL/zfs/commit/050b0e698e39768f7e3fede14536b83c93e72d82#commitcomment-17564390","id":17564390,"user":{"login":"adilger","id":167584,"avatar_url":"https://avatars.githubusercontent.com/u/167584?v=3","gravatar_id":"","url":"https://api.github.com/users/adilger","html_url":"https://github.com/adilger","followers_url":"https://api.github.com/users/adilger/followers","following_url":"https://api.github.com/users/adilger/following{/other_user}","gists_url":"https://api.github.com/users/adilger/gists{/gist_id}","starred_url":"https://api.github.com/users/adilger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/adilger/subscriptions","organizations_url":"https://api.github.com/users/adilger/orgs","repos_url":"https://api.github.com/users/adilger/repos","events_url":"https://api.github.com/users/adilger/events{/privacy}","received_events_url":"https://api.github.com/users/adilger/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":null,"commit_id":"050b0e698e39768f7e3fede14536b83c93e72d82","created_at":"2016-05-20T22:55:23Z","updated_at":"2016-05-20T22:55:23Z","body":"Patch looks very simple and straight forward, and I have only minor suggestions for improvement.\r\n\r\nI know @bzzz77 was also working on this issue, and has a more complex solution that tracks specific freed dnodes.  Alex's _might_ be faster in a heavy create/unlink workload since it can find the empty dnode slots more easily.  It might be worthwhile to compare the two, but other things being equal this patch has the benefit of being less complex and easier to understand, and less likely to have any bugs."}}
{"actor":{"display_login":"adilger"}}