{"push_id":771277155,"size":24,"distinct_size":24,"ref":"refs/heads/gateway_batch1","head":"9e639d2f0564d01628005c89687b1c11397e1f08","before":"862f73a8e20746ee5cb896db6ab9964750643868","commits":[{"sha":"3ee6abdbe5b4ab96ac7bccb14c2f91eafb969195","author":{"email":"b25b72a65195e46aa49449589ffbea257ff840a3@gmail.com","name":"Peter Mattis"},"message":"Allow parameter substition in the value of a SET statement.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/3ee6abdbe5b4ab96ac7bccb14c2f91eafb969195"},{"sha":"08fd1b0d31f8b749c726a676b0446d407ecc1069","author":{"email":"4b8373d016f277527198385ba72fda0feb5da015@cockroachlabs.com","name":"Peter Mattis"},"message":"Merge pull request #2265 from cockroachdb/pmattis/sql-set-param\n\nAllow parameter substition in the value of a SET statement.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/08fd1b0d31f8b749c726a676b0446d407ecc1069"},{"sha":"282ce0a2978db0f83632c90e51ed927a4be8819f","author":{"email":"c96a05aaafbe4b3abc1841a4083a1b0f402c2ba6@gmail.com","name":"Tamir Duberstein"},"message":"Remove `util.Ordered` which is not used","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/282ce0a2978db0f83632c90e51ed927a4be8819f"},{"sha":"1e62d03f23e5b8309863fa6cc5e0643964949392","author":{"email":"c96a05aaafbe4b3abc1841a4083a1b0f402c2ba6@gmail.com","name":"Tamir Duberstein"},"message":"Remove `gossip.InfoSlice` which is not used","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/1e62d03f23e5b8309863fa6cc5e0643964949392"},{"sha":"2120a3288b0b10ac8d49fc32a8d21644ac148aab","author":{"email":"c96a05aaafbe4b3abc1841a4083a1b0f402c2ba6@gmail.com","name":"Tamir Duberstein"},"message":"Merge pull request #2268 from tamird/remove-ordered\n\nRemove `util.Ordered` which is not used","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/2120a3288b0b10ac8d49fc32a8d21644ac148aab"},{"sha":"dd2f042e9b15dfb21a7a2dcb2055b39bc68987d7","author":{"email":"d33c80bc45d65303e33ca83108a9952b745af9ef@cockroachlabs.com","name":"marc"},"message":"Restrict KV endpoint to NodeUser only.\n\nFixes #2089.\n\nThis removes permissions for root on the kv endpoints by hard-coding the\nrequested user to security.NodeUser.\nIn insecure mode, no permissions required (previously, we defaulted to\n'root' so same thing).\nIn secure mode, you need node client certs to do anything.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/dd2f042e9b15dfb21a7a2dcb2055b39bc68987d7"},{"sha":"641d9753bac05965297c188002997da8fcb5de4a","author":{"email":"889f0264b72fd0c932d9e8e2dcc6548e982970f4@gmail.com","name":"Thomas Schroeter"},"message":"Use command line flags in sql bank example.\n\nSimilar to flag usage in the kv example.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/641d9753bac05965297c188002997da8fcb5de4a"},{"sha":"de7930eeee3bff02063738459437cfb1c8cf8e6b","author":{"email":"889f0264b72fd0c932d9e8e2dcc6548e982970f4@gmail.com","name":"Thomas Schroeter"},"message":"Add balance check interval flag.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/de7930eeee3bff02063738459437cfb1c8cf8e6b"},{"sha":"205e626cf25d38e2800f5cab815fc40df0d42761","author":{"email":"c96a05aaafbe4b3abc1841a4083a1b0f402c2ba6@gmail.com","name":"Tamir Duberstein"},"message":"Merge pull request #2269 from thschroeter/sql-bank-example-flags\n\nUse command line flags in sql bank example.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/205e626cf25d38e2800f5cab815fc40df0d42761"},{"sha":"44586540139cce38ab1b74e4f7c34d4449f191c0","author":{"email":"1fa2ef4755a9226cb9a0a4840bd89b158ac71391@cockroachlabs.com","name":"Matt Tracy"},"message":"Remove Replica.metaLock mutex\n\nFor issue #2152\nThis mutex was providing no real concurrent safety, because it gave no\nconsideration to actions by other stores. Concurrent safety is already provided\nby the transactions in each operation, so it can be removed.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/44586540139cce38ab1b74e4f7c34d4449f191c0"},{"sha":"9ffbd1a6413f444d42fbf6bca482d3eb01827b64","author":{"email":"1fa2ef4755a9226cb9a0a4840bd89b158ac71391@cockroachlabs.com","name":"Matt Tracy"},"message":"Supply RangeDescriptor to Split/Merge/Change Funcs\n\nFor #2152\n\nThis commit modifies AdminSplit, AdminMerge and ChangeReplicas to accept a\nRangeDescriptor object.\n\nThe RangeDescriptor is used as a form of optimistic locking. All of these\noperations will ultimately modify the RangeDescriptor in some way; to ensure\nthat no other concurrent operations modify the RangeDescriptor, a copy of the\noriginal RangeDescriptor is captured before the modified RangeDescriptor is\ncomputed. The original RangeDescriptor is passed to a ConditionalPut as the\nfirst operation in the transaction, causing it to fail if concurrent\nmodifications have already committed.\n\nHowever, capturing the original RangeDescriptor inside of the Split/Merge/Change\nmethods is insufficient. In most cases, the decision to call Split/Merge/Change\nis made by another method based on the RangeDescriptor, and the decision may\nhave been different if a concurrent change is applied first. Therefore, the\noriginal RangeDescriptor needs to be captured before calling Split/Merge/Change.\n\nThis commit modifies all three methods to use the new pattern, with a required\nRangeDescriptor being passed to the method.\n\nNote that the behavior of Split and Merge is still not optimal, because the\ncall to AdminSplit or AdminMerge is always routed through Raft. This pattern is\nunnecessary in cases where the split/merge decision is computed on the server.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/9ffbd1a6413f444d42fbf6bca482d3eb01827b64"},{"sha":"a29493125267ecb45a62280bf3b40845092137d6","author":{"email":"9b34bf5d02b85903d4d1681e4a5304cea6192e23@gmail.com","name":"mrtracy"},"message":"Merge pull request #2241 from mrtracy/mtracy-replica-change-safety\n\nWork towards #2152, Increasing transactional safety of replica changes","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/a29493125267ecb45a62280bf3b40845092137d6"},{"sha":"7220077abd10e20e0cabae5a9ef5340c89d5f60d","author":{"email":"da762461909883b2959497ae853551f67a91fd69@gmail.com","name":"mberhault"},"message":"Merge pull request #2264 from cockroachdb/marc/restrict_kv_to_node_user\n\nRestrict KV endpoint to NodeUser only.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/7220077abd10e20e0cabae5a9ef5340c89d5f60d"},{"sha":"07e3308f78168c503e86e5f146e58822e067c8ea","author":{"email":"d33c80bc45d65303e33ca83108a9952b745af9ef@cockroachlabs.com","name":"marc"},"message":"Fix cobra help template.\n\nSome template function names changed in\nhttps://github.com/spf13/cobra/pull/122/files which we picked up when\nsyncing.\n\nThe plain output was:\n```shell\n$ ./cockroach\nUsage:\n  cockroach [command]\n```\n\nAlso add a test for usage output. There are a few oddities about it,\nbut we want to make sure we don't lose the entire help message just\nbecause the template changed again.\n\nAlso remove trailing whitespace characters on \"Usage:\" and \"Available\nCommands:\" lines.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/07e3308f78168c503e86e5f146e58822e067c8ea"},{"sha":"bfee00966b4df6735916dc13d4324103e82d8abc","author":{"email":"da762461909883b2959497ae853551f67a91fd69@gmail.com","name":"mberhault"},"message":"Merge pull request #2270 from cockroachdb/marc/fix_cli_help\n\nFix cobra help template.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/bfee00966b4df6735916dc13d4324103e82d8abc"},{"sha":"7c5bba24b7e83d11ca1913708dbac8907afa2e8d","author":{"email":"25fad7a39062a81fcaf88abd782f9f69679c0bdf@gmail.com","name":"Tobias Schottdorf"},"message":"POC batch: DistSender.Send to Store.ExecuteCmd\n\nsee #2130, #1998 for context.\n\n* change `(*DistSender).Send` to wrap all requests in a single-\n  element Batch (TxnCoordSender unwrapped any `Batch` previously).\n* `(*Store).ExecuteCmd` unwraps this single-element `Batch`.\n* verifyPermissions: check only the inside of BatchRequest.\n* introduce `Reverse` flag on `BatchRequest` to use instead of checking for\n  `ReverseScanRequest`. Goal: whole `Batch` is either `Reverse` or not, and\n  any range-op in it is simply going to be using the reverse order. Not being\n  able to mix reverse/non-reverse requests stems from the change in descriptor\n  lookup logic.\n* implemented `proto.Combinable` for `BatchRequest`; changed it to return an\n`error`.\n* implement logic to deal with `proto.Countable` and `proto.Bounded` in `Send`.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/7c5bba24b7e83d11ca1913708dbac8907afa2e8d"},{"sha":"716f5b4c0352e7b58c56bcda6aa7bb131be53c99","author":{"email":"25fad7a39062a81fcaf88abd782f9f69679c0bdf@gmail.com","name":"Tobias Schottdorf"},"message":"batch expansion at (*Store.ExecuteCmd)\n\nupdated ExecuteCmd so that it properly unrolls a Batch.\nNote that it has to do hacky things with `Key` and `EndKey`\nto make sure they are truncated appropriately if the `BatchRequest`\nis (after all, that's all `DistSender` adapts for cross-range ops).\nOther than that, we mostly propagate the whole header from the Batch\nand the Txn (if any) from each response to the next request.\n\n`TxnCoordSender` still unrolls Batches and `(*DistSender).Send` still\nwraps each individual request in a new Batch.\n\nSome store tests use batches \"properly\", and so does `resolveWriteIntentError`,\nso the new functionality is actually tested and it took a bit to get all\ntests to pass again.\n\nNote also that the first simplification was made: the test sender in ./storage\nlost its fake batch handling.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/716f5b4c0352e7b58c56bcda6aa7bb131be53c99"},{"sha":"651de2ae2bae6992b67b2eb6ebc1c336a1fecc98","author":{"email":"25fad7a39062a81fcaf88abd782f9f69679c0bdf@gmail.com","name":"Tobias Schottdorf"},"message":"complete Is{Read,Write,...} for BatchRequest","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/651de2ae2bae6992b67b2eb6ebc1c336a1fecc98"},{"sha":"6cb1a7506c1555993a3a276b1058c1cb1b987e2b","author":{"email":"25fad7a39062a81fcaf88abd782f9f69679c0bdf@gmail.com","name":"Tobias Schottdorf"},"message":"add batch-compatible abstractions\n\nchecking for EndTransaction and intents uses a (likely temp)\nhelper that produces correct results for `BatchRequest`.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/6cb1a7506c1555993a3a276b1058c1cb1b987e2b"},{"sha":"8b158d9fe915c784c3d4fb926d2c7bd93e3ab0e3","author":{"email":"25fad7a39062a81fcaf88abd782f9f69679c0bdf@gmail.com","name":"Tobias Schottdorf"},"message":"proper truncation of batch at DistSender\n\nthis change removes the provisional truncation code in\n`(*Store).ExecuteCmd` in favor of cleaned up code in\n`(*DistSender).Send/sendAttempt`, which has also seen\nsome refactoring and many new TODOs.","distinct":true,"url":"https://api.github.com/repos/tschottdorf/cockroach/commits/8b158d9fe915c784c3d4fb926d2c7bd93e3ab0e3"}]}