{"push_id":964478177,"size":24,"distinct_size":24,"ref":"refs/heads/sql-prepare-subquery","head":"90f2048d01c81a49a94e5382232f3d1dbbdd7a26","before":"c24ddb1998b76e6f1f9203142a0f961f7debb3d8","commits":[{"sha":"e263023b1eb84cecee3544f4a44abeac7a744cac","author":{"email":"ec215cb41b22f2559be283cc58dc7219738dc864@cockroachlabs.com","name":"Bram Gruneir"},"message":"Acceptance: Update test flags","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/e263023b1eb84cecee3544f4a44abeac7a744cac"},{"sha":"61337459ea6087e6493a61bc02e28a80289a071b","author":{"email":"885ea01d97daf7784d4a0ae7f66c69a3686faeb6@gmail.com","name":"Nathan VanBenschoten"},"message":"sql/parser: Convert from DDecimal implementing Datum to *DDecimal\n\nThis should prevent allocations in cases whenever a pointer to an\nembedded `inf.Dec` is needed directly from a DDecimal by replacing:\n\n```\nfn: func(_ EvalContext, args DTuple) (Datum, error) {\n    dec := args[0].(DDecimal)\n    dd := DDecimal{}\n    dd.Round(&dec.Dec, inf.Scale(args[1].(DInt)), inf.RoundHalfUp)\n    return dd, nil\n}\n```\n\nwith\n\n```\nfn: func(_ EvalContext, args DTuple) (Datum, error) {\n   dec := args[0].(*DDecimal)\n   dd := &DDecimal{}\n   dd.Round(&dec.Dec, inf.Scale(args[1].(DInt)), inf.RoundHalfUp)\n   return dd, nil\n}\n```","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/61337459ea6087e6493a61bc02e28a80289a071b"},{"sha":"b1ba09c2b723791b4060b02e16d0d030255802ca","author":{"email":"b25b72a65195e46aa49449589ffbea257ff840a3@gmail.com","name":"Peter Mattis"},"message":"Make *UnresolvedAddr implement net.Addr, not UnresolvedAddr.\n\nThis is done because assigning a non-empty struct to an interface requires an\nallocation, while assigning a pointer to an interface is allocation free. Using\nan *UnresolvedAddr makes it both clear that an allocation is occurring and\nallows us to avoid an allocation when an UnresolvedAddr is a field of a\nstruct (e.g. NodeDescriptor.Address). This saves a few allocations on every\nDistSender RPC.","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/b1ba09c2b723791b4060b02e16d0d030255802ca"},{"sha":"208bc8adfdac1f42d4a85f12666cba96e44927f4","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@cockroachlabs.com","name":"Peter Mattis"},"message":"Merge pull request #4131 from petermattis/pmattis/unresolved-addr\n\nMake *UnresolvedAddr implement net.Addr, not UnresolvedAddr.","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/208bc8adfdac1f42d4a85f12666cba96e44927f4"},{"sha":"6e96cfa5330a363de05ff87e8b8b58275b3a5fd2","author":{"email":"b25b72a65195e46aa49449589ffbea257ff840a3@gmail.com","name":"Peter Mattis"},"message":"Fix random shuffle.\n\nWe were failing to shuffle the last entry.","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/6e96cfa5330a363de05ff87e8b8b58275b3a5fd2"},{"sha":"eabbafb9ca37d933540119f2880eeb3cc1057fc4","author":{"email":"2f2f7f3d3dbf2eca4c6ade784c166610bf59388f@users.noreply.github.com","name":"Bram Gruneir"},"message":"Merge pull request #4119 from BramGruneir/testconfig\n\nAcceptance: Update test flags","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/eabbafb9ca37d933540119f2880eeb3cc1057fc4"},{"sha":"8100b7b8dcf63d425a51667f7c323ce268cb8e01","author":{"email":"885ea01d97daf7784d4a0ae7f66c69a3686faeb6@gmail.com","name":"Nathan VanBenschoten"},"message":"Merge pull request #4116 from nvanbenschoten/nvanbenschoten/decptr\n\nsql/parser: Convert from DDecimal implementing Datum to *DDecimal","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/8100b7b8dcf63d425a51667f7c323ce268cb8e01"},{"sha":"3f064f6706db5e6477bd486e2fd890a9bc52ce08","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@cockroachlabs.com","name":"Peter Mattis"},"message":"Merge pull request #4132 from petermattis/pmattis/fix-shuffle\n\nFix random shuffle.","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/3f064f6706db5e6477bd486e2fd890a9bc52ce08"},{"sha":"939c0d7e7fdbc2f5171f0b3bfe5ef57a83fe9fe6","author":{"email":"45aefc834abf4b6844854d63a474b67e53b69671@gmail.com","name":"Spencer Kimball"},"message":"Protect against dev mode instances attempting to connect to a cluster\n\nAlso added a hard-to-miss notice when running in --dev mode.","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/939c0d7e7fdbc2f5171f0b3bfe5ef57a83fe9fe6"},{"sha":"c63785ecce051a62ea2f60e1fa8516f4a7d8a6a5","author":{"email":"73675debcd8a436be48ec22211dcf44fe0df0a64@cockroachlabs.com","name":"Ben Darnell"},"message":"cli: Add commands to print useful information from store data files.\n\nMotivated by investigations in #3896.","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/c63785ecce051a62ea2f60e1fa8516f4a7d8a6a5"},{"sha":"87ec0a5b1c96f983453369905b377a2392699ab1","author":{"email":"73675debcd8a436be48ec22211dcf44fe0df0a64@bendarnell.com","name":"Ben Darnell"},"message":"Merge pull request #4123 from bdarnell/cli-debug\n\ncli: Add commands to print useful information from store data files.","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/87ec0a5b1c96f983453369905b377a2392699ab1"},{"sha":"3d691f4e392918332ff8cfa3f46abb8cdcc8ee2f","author":{"email":"b00247e2d759197c8b2da85e3eb4533fb49f53f7@gmail.com","name":"Vivek Menezes"},"message":"cleanup PGUrl","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/3d691f4e392918332ff8cfa3f46abb8cdcc8ee2f"},{"sha":"03922a45a655d0972825cb398729293d0caa0537","author":{"email":"e0cb593f5504fb19339078306a174cb6c8837801@users.noreply.github.com","name":"vivekmenezes"},"message":"Merge pull request #4115 from vivekmenezes/vivek/url\n\ncleanup PGUrl","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/03922a45a655d0972825cb398729293d0caa0537"},{"sha":"627a94d3eeb61a58650aa0b8e69d97b14d90a4de","author":{"email":"ae685575101ee7165c90a8f2c30c6e60cdd9e482@lightyear.no","name":"Roy Hvaara"},"message":"Added two recent talks and updated timestamp on an old one","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/627a94d3eeb61a58650aa0b8e69d97b14d90a4de"},{"sha":"002402c4a147c87a0895c676e0a93492f321f191","author":{"email":"b00247e2d759197c8b2da85e3eb4533fb49f53f7@gmail.com","name":"Vivek Menezes"},"message":"use bank example in chaos test","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/002402c4a147c87a0895c676e0a93492f321f191"},{"sha":"a3df9481336ecaa8e56aa6c88172f96008ef3fca","author":{"email":"45aefc834abf4b6844854d63a474b67e53b69671@gmail.com","name":"Spencer Kimball"},"message":"Merge pull request #4135 from cockroachdb/spencerkimball/fix-dev-mode\n\nprotect against dev mode instances attempting to connect to a cluster","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/a3df9481336ecaa8e56aa6c88172f96008ef3fca"},{"sha":"4eeb73fa819852fc09b828cba13b8b6a12e64708","author":{"email":"e0cb593f5504fb19339078306a174cb6c8837801@users.noreply.github.com","name":"vivekmenezes"},"message":"Merge pull request #4124 from vivekmenezes/vivek/bank\n\nuse bank example in chaos test","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/4eeb73fa819852fc09b828cba13b8b6a12e64708"},{"sha":"691206bfed6a4acffeab5160b4c55b1b4ae02036","author":{"email":"c96a05aaafbe4b3abc1841a4083a1b0f402c2ba6@gmail.com","name":"Tamir Duberstein"},"message":"Merge pull request #4143 from hvaara/update-readme\n\nAdded two recent talks to README.md","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/691206bfed6a4acffeab5160b4c55b1b4ae02036"},{"sha":"dc102068f77d1e0a8f2397ba977315898cdc6184","author":{"email":"885ea01d97daf7784d4a0ae7f66c69a3686faeb6@gmail.com","name":"Nathan VanBenschoten"},"message":"sql: Create a suite of benchmarks for aggregation functions\n\nCurrent results are:\n\n```\nBenchmarkAvgAggregateInt-4\t\t   20000\t     68674 ns/op\nBenchmarkAvgAggregateFloat-4\t\t   20000\t     70410 ns/op\nBenchmarkAvgAggregateDecimal-4\t\t    3000\t    397697 ns/op\nBenchmarkCountAggregate-4\t\t  200000\t      6874 ns/op\nBenchmarkSumAggregateInt-4\t\t   20000\t     65337 ns/op\nBenchmarkSumAggregateFloat-4\t\t   20000\t     68260 ns/op\nBenchmarkSumAggregateDecimal-4\t\t    3000\t    390299 ns/op\nBenchmarkMaxAggregateInt-4\t\t  100000\t     17648 ns/op\nBenchmarkMaxAggregateFloat-4\t\t  100000\t     17894 ns/op\nBenchmarkMaxAggregateDecimal-4\t\t   10000\t    157491 ns/op\nBenchmarkMinAggregateInt-4\t\t  100000\t     17654 ns/op\nBenchmarkMinAggregateFloat-4\t\t  100000\t     17517 ns/op\nBenchmarkMinAggregateDecimal-4\t\t    5000\t    295761 ns/op\nBenchmarkVarianceAggregateInt-4\t\t    1000\t   1762158 ns/op\nBenchmarkVarianceAggregateFloat-4\t   20000\t     75138 ns/op\nBenchmarkVarianceAggregateDecimal-4\t    1000\t   1650435 ns/op\nBenchmarkStddevAggregateInt-4\t\t    1000\t   1963264 ns/op\nBenchmarkStddevAggregateFloat-4\t\t   20000\t     77677 ns/op\nBenchmarkStddevAggregateDecimal-4\t    1000\t   1667530 ns/op\n```","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/dc102068f77d1e0a8f2397ba977315898cdc6184"},{"sha":"b5948ff359c733ae516f3d6c074b6fbe325c9af4","author":{"email":"314fe2776746d1cb8b8611e1e315a38b2017bbff@gmail.com","name":"Matt Jibson"},"message":"sql: support prepare for CASE statements\n\nAlso add protecting against other statements where valargs are not fully\nsupported. Say that $2 gets inferred but $1 does not. The previous code\nwould panic because the slice isn't long enough (expected to be of length\n2 when it indexed into pq.inTypes[1]) because it only made pq.inTypes\nlength 1 since that's how many valargs it inferred.\n\nFixes #4024","distinct":true,"url":"https://api.github.com/repos/mjibson/cockroach/commits/b5948ff359c733ae516f3d6c074b6fbe325c9af4"}]}