{"action":"created","issue":{"url":"https://api.github.com/repos/greenplum-db/gpdb/issues/731","repository_url":"https://api.github.com/repos/greenplum-db/gpdb","labels_url":"https://api.github.com/repos/greenplum-db/gpdb/issues/731/labels{/name}","comments_url":"https://api.github.com/repos/greenplum-db/gpdb/issues/731/comments","events_url":"https://api.github.com/repos/greenplum-db/gpdb/issues/731/events","html_url":"https://github.com/greenplum-db/gpdb/pull/731","id":154008609,"number":731,"title":"Mapred musings","user":{"login":"danielgustafsson","id":321790,"avatar_url":"https://avatars.githubusercontent.com/u/321790?v=3","gravatar_id":"","url":"https://api.github.com/users/danielgustafsson","html_url":"https://github.com/danielgustafsson","followers_url":"https://api.github.com/users/danielgustafsson/followers","following_url":"https://api.github.com/users/danielgustafsson/following{/other_user}","gists_url":"https://api.github.com/users/danielgustafsson/gists{/gist_id}","starred_url":"https://api.github.com/users/danielgustafsson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danielgustafsson/subscriptions","organizations_url":"https://api.github.com/users/danielgustafsson/orgs","repos_url":"https://api.github.com/users/danielgustafsson/repos","events_url":"https://api.github.com/users/danielgustafsson/events{/privacy}","received_events_url":"https://api.github.com/users/danielgustafsson/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"danielgustafsson","id":321790,"avatar_url":"https://avatars.githubusercontent.com/u/321790?v=3","gravatar_id":"","url":"https://api.github.com/users/danielgustafsson","html_url":"https://github.com/danielgustafsson","followers_url":"https://api.github.com/users/danielgustafsson/followers","following_url":"https://api.github.com/users/danielgustafsson/following{/other_user}","gists_url":"https://api.github.com/users/danielgustafsson/gists{/gist_id}","starred_url":"https://api.github.com/users/danielgustafsson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danielgustafsson/subscriptions","organizations_url":"https://api.github.com/users/danielgustafsson/orgs","repos_url":"https://api.github.com/users/danielgustafsson/repos","events_url":"https://api.github.com/users/danielgustafsson/events{/privacy}","received_events_url":"https://api.github.com/users/danielgustafsson/received_events","type":"User","site_admin":false},"milestone":null,"comments":8,"created_at":"2016-05-10T13:38:43Z","updated_at":"2016-05-18T01:07:52Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/greenplum-db/gpdb/pulls/731","html_url":"https://github.com/greenplum-db/gpdb/pull/731","diff_url":"https://github.com/greenplum-db/gpdb/pull/731.diff","patch_url":"https://github.com/greenplum-db/gpdb/pull/731.patch"},"body":"Handle implicit casting between text and integer in the mapred test suite. The mapred code assumes text datatype for everything unless the parameter types are explicitly stated so mark the parameters to the sum function as integer. This is a test that started failing in the merge branch due to the removal of implicit casts but since the fix applies to master just as well we can fix it here and reduce the merge footprint. Further, the agebracket.yml mapred test case is testing the reduction with a simple reducer summing the values. GPMapred has a builtin SUM reducer however which will yield the same result, extend the test with a copy of the agebracket.yml test using the builtin ensuring that the output is identical.\r\n\r\nWhile I was reading the code I also fixed lots and lots of trailing whitespace issues and a few spelling errors which are included in this PR. Reading the PR commit by commit is recommended."},"comment":{"url":"https://api.github.com/repos/greenplum-db/gpdb/issues/comments/219898704","html_url":"https://github.com/greenplum-db/gpdb/pull/731#issuecomment-219898704","issue_url":"https://api.github.com/repos/greenplum-db/gpdb/issues/731","id":219898704,"user":{"login":"ascherbaum-pivotal","id":5319526,"avatar_url":"https://avatars.githubusercontent.com/u/5319526?v=3","gravatar_id":"","url":"https://api.github.com/users/ascherbaum-pivotal","html_url":"https://github.com/ascherbaum-pivotal","followers_url":"https://api.github.com/users/ascherbaum-pivotal/followers","following_url":"https://api.github.com/users/ascherbaum-pivotal/following{/other_user}","gists_url":"https://api.github.com/users/ascherbaum-pivotal/gists{/gist_id}","starred_url":"https://api.github.com/users/ascherbaum-pivotal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ascherbaum-pivotal/subscriptions","organizations_url":"https://api.github.com/users/ascherbaum-pivotal/orgs","repos_url":"https://api.github.com/users/ascherbaum-pivotal/repos","events_url":"https://api.github.com/users/ascherbaum-pivotal/events{/privacy}","received_events_url":"https://api.github.com/users/ascherbaum-pivotal/received_events","type":"User","site_admin":false},"created_at":"2016-05-18T01:07:52Z","updated_at":"2016-05-18T01:07:52Z","body":"ok, go for it,"}}