{"action":"created","issue":{"url":"https://api.github.com/repos/prestodb/presto/issues/4040","repository_url":"https://api.github.com/repos/prestodb/presto","labels_url":"https://api.github.com/repos/prestodb/presto/issues/4040/labels{/name}","comments_url":"https://api.github.com/repos/prestodb/presto/issues/4040/comments","events_url":"https://api.github.com/repos/prestodb/presto/issues/4040/events","html_url":"https://github.com/prestodb/presto/issues/4040","id":118982570,"number":4040,"title":"Improve type of hash_code operator and combine_hash function","user":{"login":"cberner","id":842688,"avatar_url":"https://avatars.githubusercontent.com/u/842688?v=3","gravatar_id":"","url":"https://api.github.com/users/cberner","html_url":"https://github.com/cberner","followers_url":"https://api.github.com/users/cberner/followers","following_url":"https://api.github.com/users/cberner/following{/other_user}","gists_url":"https://api.github.com/users/cberner/gists{/gist_id}","starred_url":"https://api.github.com/users/cberner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cberner/subscriptions","organizations_url":"https://api.github.com/users/cberner/orgs","repos_url":"https://api.github.com/users/cberner/repos","events_url":"https://api.github.com/users/cberner/events{/privacy}","received_events_url":"https://api.github.com/users/cberner/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/prestodb/presto/labels/intermediate-task","name":"intermediate-task","color":"bfe5bf"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2015-11-26T06:43:12Z","updated_at":"2016-07-13T20:26:43Z","closed_at":null,"body":"The type of hash_code and combine_hash is declared to be `BIGINT`. However, there are lots of assumptions in the code base that the value returned from these functions will be within the range of an `int`. We either need to remove these assumptions, or change the type to `INT` (and implement an `INT` type)"},"comment":{"url":"https://api.github.com/repos/prestodb/presto/issues/comments/232476225","html_url":"https://github.com/prestodb/presto/issues/4040#issuecomment-232476225","issue_url":"https://api.github.com/repos/prestodb/presto/issues/4040","id":232476225,"user":{"login":"xerial","id":57538,"avatar_url":"https://avatars.githubusercontent.com/u/57538?v=3","gravatar_id":"","url":"https://api.github.com/users/xerial","html_url":"https://github.com/xerial","followers_url":"https://api.github.com/users/xerial/followers","following_url":"https://api.github.com/users/xerial/following{/other_user}","gists_url":"https://api.github.com/users/xerial/gists{/gist_id}","starred_url":"https://api.github.com/users/xerial/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/xerial/subscriptions","organizations_url":"https://api.github.com/users/xerial/orgs","repos_url":"https://api.github.com/users/xerial/repos","events_url":"https://api.github.com/users/xerial/events{/privacy}","received_events_url":"https://api.github.com/users/xerial/received_events","type":"User","site_admin":false},"created_at":"2016-07-13T20:26:43Z","updated_at":"2016-07-13T20:26:43Z","body":"@cberner \r\nA problem related this is `SELECT rank() OVER(PARTITION_BY id)` creates inaccurate windows if combine_hash(id) has collisions in 32-bit `int` space. "}}
{"actor":{"display_login":"xerial"}}