{"comment":{"url":"https://api.github.com/repos/nileema/presto/comments/12283737","html_url":"https://github.com/nileema/presto/commit/42add03b00b1fe2fa4006c30b8248239c5dc088d#commitcomment-12283737","id":12283737,"user":{"login":"dain","id":58298,"avatar_url":"https://avatars.githubusercontent.com/u/58298?v=3","gravatar_id":"","url":"https://api.github.com/users/dain","html_url":"https://github.com/dain","followers_url":"https://api.github.com/users/dain/followers","following_url":"https://api.github.com/users/dain/following{/other_user}","gists_url":"https://api.github.com/users/dain/gists{/gist_id}","starred_url":"https://api.github.com/users/dain/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dain/subscriptions","organizations_url":"https://api.github.com/users/dain/orgs","repos_url":"https://api.github.com/users/dain/repos","events_url":"https://api.github.com/users/dain/events{/privacy}","received_events_url":"https://api.github.com/users/dain/received_events","type":"User","site_admin":false},"position":106,"line":162,"path":"presto-spi/src/main/java/com/facebook/presto/spi/block/LazySliceArrayBlock.java","commit_id":"42add03b00b1fe2fa4006c30b8248239c5dc088d","created_at":"2015-07-21T20:04:17Z","updated_at":"2015-07-21T20:04:17Z","body":"It might be easier to read if instead of having `getValue` you have:\r\n```\r\nprivate int getPosition(int position) \r\n{\r\n     assureLoaded();\r\n     if (dictionary) {\r\n         return ids[position];\r\n     }\r\n     return position;\r\n}\r\n```\r\n"}}