{"push_id":1205674688,"size":5,"distinct_size":5,"ref":"refs/heads/sprint-35","head":"2f4a9511a303f2d5386d356f3d2df42ab40cca7e","before":"094bfdac139b02906263d332211b2553cbe3327c","commits":[{"sha":"15fceb0701dbd3d0fa2c9e571f0d3cfb638b500d","author":{"name":"Andrzej Fiedukowicz","email":"2a5ec98a5003e2a3416a11ded3e31f81c769e29a@teradata.com"},"message":"Fix problem with `round` on short int types casts\n\nMath functions defines multiple versions of two argument\n`round` function:\n 1) `round(TINYINT, BIGINT)`\n 2) `round(SMALLINT, BIGINT)`\n 3) `round(INTEGER, INTEGER)`\n 4) `round(BIGINT, BIGINT)`\n\nThere is an incosequence in (3), as the second argument is\nsuprisingly INTEGER. This was only a small problem before\n`TINYINT` and `SMALLINT` were introduced.\n\nRight now it causes a problem with coersions when for\nexample `round(TINYINT, TINYINT)` is called.\nIn such scenario both (3) and (4) are valid coersions\nand so such call is ambiguous (causes an error).\n\nTo fix this we may either define (generate) 16 function\nfor all possible combination or just make second argument\n`BIGINT` for all cases (which is proposed change).\n\nThe proposed solution has a drawback of calling cast\nin some cases, but this cast is noop in current implementation.\nThe return type with 4 functions is determined correctly only\nbased on first argument type.\n\nThis commit also introduces test for smaller integer type\nfor `round` function which do cover introduced change.","distinct":true,"url":"https://api.github.com/repos/Teradata/presto/commits/15fceb0701dbd3d0fa2c9e571f0d3cfb638b500d"},{"sha":"3d6c94b67ef6894ca3180f2f918e1d5c88cb49b6","author":{"name":"Andrzej Fiedukowicz","email":"2a5ec98a5003e2a3416a11ded3e31f81c769e29a@teradata.com"},"message":"Add signum tests for `SMALLINT` and `TINYINT`\n\nThose test were missing before.","distinct":true,"url":"https://api.github.com/repos/Teradata/presto/commits/3d6c94b67ef6894ca3180f2f918e1d5c88cb49b6"},{"sha":"7d5eda3417155d0c7b9eac31a8d5daa2fc58a306","author":{"name":"Andrzej Fiedukowicz","email":"2a5ec98a5003e2a3416a11ded3e31f81c769e29a@teradata.com"},"message":"Fix presto-server-rpm output name","distinct":true,"url":"https://api.github.com/repos/Teradata/presto/commits/7d5eda3417155d0c7b9eac31a8d5daa2fc58a306"},{"sha":"c43a5ef6777f3e3d218171e2c9e2748283af367a","author":{"name":"Andrzej Fiedukowicz","email":"2a5ec98a5003e2a3416a11ded3e31f81c769e29a@teradata.com"},"message":"Unquarantine 'show partitions' product test\n\nIt was moved from convention based test to Java based\nas convention based test don't have enough expression power\nto cover the test requirements.","distinct":true,"url":"https://api.github.com/repos/Teradata/presto/commits/c43a5ef6777f3e3d218171e2c9e2748283af367a"},{"sha":"2f4a9511a303f2d5386d356f3d2df42ab40cca7e","author":{"name":"Andrzej Fiedukowicz","email":"2a5ec98a5003e2a3416a11ded3e31f81c769e29a@teradata.com"},"message":"Unqarantine tests requiring exact number of rows inserted","distinct":true,"url":"https://api.github.com/repos/Teradata/presto/commits/2f4a9511a303f2d5386d356f3d2df42ab40cca7e"}]}
{"actor":{"display_login":"fiedukow"}}