{"push_id":957129692,"size":11,"distinct_size":11,"ref":"refs/heads/master","head":"cacaa1de882fab009581a3a179ea1e22c8b0281c","before":"d264ca55087e3c1587096176341f92f45268ec9e","commits":[{"sha":"87ed427273c9dc52c7e6e7a69d616dd99f331e82","author":{"email":"b5323f9c6bd5e9b9016693ec73c8378c29769616@teradata.com","name":"Piotr Nowojski"},"message":"Re-enable assertion in TestExpressionCompiler\n\nIt was disabled due to missing functionality.","distinct":true,"url":"https://api.github.com/repos/facebook/presto/commits/87ed427273c9dc52c7e6e7a69d616dd99f331e82"},{"sha":"85446dd60751409efa569aac759a788067e3da40","author":{"email":"b5323f9c6bd5e9b9016693ec73c8378c29769616@teradata.com","name":"Piotr Nowojski"},"message":"Implement VARCHAR(length)\n\n1. Implement VARCHAR(length)\n\n2. Add mini-language to calculate return type parameter\nco-authored with Dain Sundstrom\n\n3. Insert cast function even for type-only coercions\n\nThis patch changes the way optimization for not inserting cast calls for\ntype-type only coercions work.\n\nOriginal code caused errors during bytecode preparation for plan execution.\nProblematic case:\n\nARRAY('dog', 'kitten').\nDuring semantic analysis this was treated as:\nARRAY((CAST 'dog' as VARCHAR(6)), 'kitten') and therefore\nimplementation of ArrayConstructor for parameter type VARCHAR(6) was\ngenerated.\n\nBut cast expression was not added to result query expression, as\ncase VARCHAR(3) -> VARCHAR(6) is type-only cast.\n\nThen byte generating code was failing as it was looking for\ninstantiation of ARRAY_CONSTRUCTOR(VARCHAR(3), VARCHAR(6)) despite the fact\nonly ARRAY_CONSTRUCTOR(VARCHAR(6), VARCHAR(6)) was generated.\n\nNow we add Cast operator to AST. But it has typeOnly marker set to true.\nAnd then when execution plan is generated no cast operation call is\ngenerated.\nco-authored with Lukas Osipiuk\n\n4. Add explicit literalParameters","distinct":true,"url":"https://api.github.com/repos/facebook/presto/commits/85446dd60751409efa569aac759a788067e3da40"},{"sha":"4845826639f863de22d43c5f5fae3daa85226d79","author":{"email":"b5323f9c6bd5e9b9016693ec73c8378c29769616@teradata.com","name":"Piotr Nowojski"},"message":"Add test for calling varchar(x) function with varchar","distinct":true,"url":"https://api.github.com/repos/facebook/presto/commits/4845826639f863de22d43c5f5fae3daa85226d79"},{"sha":"02f4b461c904d859bb4123b39e887ad425c66621","author":{"email":"b5323f9c6bd5e9b9016693ec73c8378c29769616@teradata.com","name":"Piotr Nowojski"},"message":"Add test for matching UNKNOWN argument to VARCHAR(x) function","distinct":true,"url":"https://api.github.com/repos/facebook/presto/commits/02f4b461c904d859bb4123b39e887ad425c66621"},{"sha":"d165b72cb7df4bbe4ac35ba5bd03a41db1369f1a","author":{"email":"b5323f9c6bd5e9b9016693ec73c8378c29769616@teradata.com","name":"Piotr Nowojski"},"message":"Add more comprehensive tests for Signature","distinct":true,"url":"https://api.github.com/repos/facebook/presto/commits/d165b72cb7df4bbe4ac35ba5bd03a41db1369f1a"},{"sha":"30fa0bd3bf73e38b67cd66e48d3975dbfbee4676","author":{"email":"f89a08b50808f40d497bf3206dda6c9147dd6b3f@Teradata.com","name":"Karol Sobczak"},"message":"Add support for parametric Varchar to hive connector\n\nAllow inserting varchar(n) into a varchar(m) column as long as n <= m.","distinct":true,"url":"https://api.github.com/repos/facebook/presto/commits/30fa0bd3bf73e38b67cd66e48d3975dbfbee4676"},{"sha":"c732d7edfed2c5850043ed21c20f5c58a1192972","author":{"email":"6c5eb78e54fa6aff67ba440c4bd5b1e43956c922@osipiuk.net","name":"Lukasz Osipiuk"},"message":"Relax INSERT type validation constraints\n\nThis is a workaround for implicit coercions not being\nsupported for INSERT queries.\n\n1. relax limitation that when INSERT INTO statement is called\nvalues/query types must exactly match table column types.\nNow we allow values/query types to be type-only coercible to table\ncolumn types.\n\n2. allow difference between inserted values and table column types if\nthey are type-coercable for structural types.","distinct":true,"url":"https://api.github.com/repos/facebook/presto/commits/c732d7edfed2c5850043ed21c20f5c58a1192972"},{"sha":"74633326b06aa612b66aae1dbae57383c6f72b7d","author":{"email":"b5323f9c6bd5e9b9016693ec73c8378c29769616@teradata.com","name":"Piotr Nowojski"},"message":"Verify types have the same number of parameters\n\nWe want to keep logic of TypeRegistry#getCommonSuperTypeSignature as generic as possible.\nTherefore we add assertion in getCommonSuperTypeSignature disallowing\ntypesignature without expanded implicit literal arguments.\n\nIn running presto it should never happen, that data type without implicit literal arguments\nexpanded, is passed as argument to getCommonSuperTypeSignature.","distinct":true,"url":"https://api.github.com/repos/facebook/presto/commits/74633326b06aa612b66aae1dbae57383c6f72b7d"},{"sha":"f2ea764f43d07563592c988dde9bd6b5dc4e436c","author":{"email":"b5323f9c6bd5e9b9016693ec73c8378c29769616@teradata.com","name":"Piotr Nowojski"},"message":"Rename ParameterKind values","distinct":true,"url":"https://api.github.com/repos/facebook/presto/commits/f2ea764f43d07563592c988dde9bd6b5dc4e436c"},{"sha":"56cbe7e44340f4ed696a0292b6aefc422a2835f5","author":{"email":"6c5eb78e54fa6aff67ba440c4bd5b1e43956c922@osipiuk.net","name":"Lukasz Osipiuk"},"message":"Use new syntax for defining type parameters\n\nReplace all deprecated angle bracket based syntax with new\nparentheses based.","distinct":true,"url":"https://api.github.com/repos/facebook/presto/commits/56cbe7e44340f4ed696a0292b6aefc422a2835f5"},{"sha":"cacaa1de882fab009581a3a179ea1e22c8b0281c","author":{"email":"a3f23a76a6203525fe5c769a7a3ec65aa430adc3@gmail.com","name":"Martin Traverso"},"message":"Render varchar(max) as varchar","distinct":true,"url":"https://api.github.com/repos/facebook/presto/commits/cacaa1de882fab009581a3a179ea1e22c8b0281c"}]}