{"push_id":751222358,"size":23,"distinct_size":23,"ref":"refs/heads/master","head":"57b0d5015ac901fd5260af481bad25ca7c54d703","before":"53c6ff6e47743f487b1d3aceedb38e36c445be06","commits":[{"sha":"9aec23bd28ecaac6777cdb08bcc42e3d94928845","author":{"email":"e9183a7ba5c69cc060de2ed3bf6b5a2ee7921775@vendasta.com","name":"Kevin Sookocheff"},"message":"Fix common test runner.","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/9aec23bd28ecaac6777cdb08bcc42e3d94928845"},{"sha":"1db442451e9f4a93684d3f9d3224ead4eab7c16b","author":{"email":"be5acd4cab620d494846b6383e5e26dbab0e82f4@khanacademy.org","name":"Craig Silverstein"},"message":"Support reading the data files even when mapreduce is in a zipfile.\n\nSummary:\nzipimport makes it possible to read python files from a .zip, but it\ndoesn't address reading data files.  The code has to be written to\nsupport that.  Change the mapreduce code so it is.  Luckily it's\npretty localized.\n\nDon't assume that zipfiles end with .zip, and .zip-things are zipfiles.\n\nJust let the zipfile module tell us whether something is a zipfile or not.\n\nTest Plan:\nDid a non-standard deploy and visited the following urls and made sure\nthey gave back non-error content:\n   http://znd-zipimport.khan-academy.appspot.com/mapreduce/status\n   http://znd-zipimport.khan-academy.appspot.com/mapreduce/status.js\n   http://znd-zipimport.khan-academy.appspot.com/mapreduce/jquery.js\n   http://znd-zipimport.khan-academy.appspot.com/mapreduce/lib/pipeline/status.js\n   http://znd-zipimport.khan-academy.appspot.com/mapreduce/lib/pipeline/images/treeview-black.gif\n\nReviewers: alpert, kamens\n\nReviewed By: alpert\n\nDifferential Revision: http://phabricator.khanacademy.org/D884","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/1db442451e9f4a93684d3f9d3224ead4eab7c16b"},{"sha":"a30e755f47c0216757a99d98ec92066996a4fdbb","author":{"email":"3298d1f1b2df22b8eb61ea39c8d987f00677c748@khanacademy.org","name":"MattFaus"},"message":"Fix /mapreduce/pipeline to properly ZipImport\n\nTest Plan:\nCraig's review\n(ZipImport's only happen in production, so hard to test locally)\n\nReviewers: csilvers\n\nDifferential Revision: http://phabricator.khanacademy.org/D2281","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/a30e755f47c0216757a99d98ec92066996a4fdbb"},{"sha":"1137cb924525bb26c92de31114c2d3b4148135ad","author":{"email":"91e38e63b890fbb214c8914809fde03c73e7f24d@khanacademy.org","name":"Alan Pierce"},"message":"Disable InOrder when running pipelines in test mode\n\nSummary:\nInOrder uses a thread-local to make sure you don't have an InOrder nested within\nanother InOrder, which works normally because each pipeline is run in a separate\nprocess. In test mode, though, everything is run in one thread, so this\nreentrancy check incorrectly throws an exception. Since InOrder has no effect in\ntest mode anyway, I fixed this by just disabling InOrder for tests.\n\nTest Plan:\nThis was breaking a new test in webapp for some code that had been running\ncorrectly in production mode for a while. With this commit, the test works.\n\nReviewers: mattfaus\n\nReviewed By: mattfaus\n\nDifferential Revision: http://phabricator.khanacademy.org/D7815","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/1137cb924525bb26c92de31114c2d3b4148135ad"},{"sha":"32fa1cb2ffa4b861a17c90c594b8a094a97f5597","author":{"email":"91e38e63b890fbb214c8914809fde03c73e7f24d@khanacademy.org","name":"Alan Pierce"},"message":"Remove an incorrect assertion in pipeline test mode\n\nSummary:\nThe implementation of evaluate_test had an assertion that all outputs of a\npipeline are always consumed by the calling pipeline, but as far as I can tell,\nthis isn't enforced for regular production mode, so maybe the test assertion was\nleft in by accident. It's breaking a test on a pipeline that has been running\nfine for a while, so I'm removing the test sanity check.\n\nTest Plan: Run summarize_fast_ete_test in webapp, which ran into this bug.\n\nReviewers: mattfaus\n\nReviewed By: mattfaus\n\nSubscribers: benkraft, chris\n\nDifferential Revision: http://phabricator.khanacademy.org/D10679","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/32fa1cb2ffa4b861a17c90c594b8a094a97f5597"},{"sha":"d299ed6a143455e62306f059904ed6340d2273af","author":{"email":"90de8fe81e08b7daaa5a341ba307c30477b6329d@gmail.com","name":"Maciek Ruckgaber"},"message":"Adds ndb.Key JSON serialization\n\nndb JSON serialization consistency between mapreduce and pipelines. Uses\ncode commited in mapreduce.json_util by soundofjw.","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/d299ed6a143455e62306f059904ed6340d2273af"},{"sha":"d7999d5191a18f88d8704e5a6d103007c04cb998","author":{"email":"c028c213ed5efcf30c3f4fc7361dbde0c893c5b7@loudr.fm","name":"Josh Whelchel"},"message":"Merge pull request #41 from maciekrb/master\n\nAdds ndb.Key JSON serialization","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/d7999d5191a18f88d8704e5a6d103007c04cb998"},{"sha":"be1cc069b4bdca2958865567958f9f335ea788e6","author":{"email":"c028c213ed5efcf30c3f4fc7361dbde0c893c5b7@loudr.fm","name":"Josh Whelchel"},"message":"fixes #43 by supplying default bucket name","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/be1cc069b4bdca2958865567958f9f335ea788e6"},{"sha":"42f6b6eedb0f65d306ae69373e76f3ed848a8b5d","author":{"email":"5f10133efc1babfe554e2c0586806b717e1f8787@gmail.com","name":"aozarov"},"message":"provide a way to configure base url","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/42f6b6eedb0f65d306ae69373e76f3ed848a8b5d"},{"sha":"461e8deb1ca2cab4ec50a1c60a908596d4b4909b","author":{"email":"5f10133efc1babfe554e2c0586806b717e1f8787@gmail.com","name":"aozarov"},"message":"fix unicode issue","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/461e8deb1ca2cab4ec50a1c60a908596d4b4909b"},{"sha":"0dd2d2d0af4ce2ae7ee10eb59c0c0b0f0a2cfae7","author":{"email":"ca56a5f347880579ad16e22284df28d13a310849@gmail.com","name":"Tom Kaitchuck"},"message":"Merge pull request #45 from aozarov/master\n\nprovide a way to configure base url","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/0dd2d2d0af4ce2ae7ee10eb59c0c0b0f0a2cfae7"},{"sha":"8ba92cb9498620d0dd1e4b29184f4046fd10d77e","author":{"email":"5f10133efc1babfe554e2c0586806b717e1f8787@users.noreply.github.com","name":"Arie"},"message":"Revert \"added method to job to get PromiseValue using handle\"","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/8ba92cb9498620d0dd1e4b29184f4046fd10d77e"},{"sha":"8e37a5e4c570ad2473ec3791366ad29ba30b40e1","author":{"email":"5f10133efc1babfe554e2c0586806b717e1f8787@users.noreply.github.com","name":"Arie"},"message":"Merge pull request #46 from GoogleCloudPlatform/revert-14-master\n\nRevert \"added method to job to get PromiseValue using handle\"","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/8e37a5e4c570ad2473ec3791366ad29ba30b40e1"},{"sha":"7749346fa526bbe806dbe5bcd0f04a6658abf2a6","author":{"email":"c028c213ed5efcf30c3f4fc7361dbde0c893c5b7@loudr.fm","name":"Josh Whelchel"},"message":"throw exception on 'no default bucket' state\n\n`app_identity.get_default_gcs_bucket_name()` returns `None` when no\ndefault bucket is set.\n\nThis should only affect apps created before appengine release 1.9.0.","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/7749346fa526bbe806dbe5bcd0f04a6658abf2a6"},{"sha":"d6078c7032eb5e0560d2e321f61df0c560e51f98","author":{"email":"7433942b0d3e421de064bc97468c95aefcea91e6@google.com","name":"tkaitchuck"},"message":"Add build.sh to allow for running the tests easily.","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/d6078c7032eb5e0560d2e321f61df0c560e51f98"},{"sha":"2561501271f510c89db9dd2e9de1691eb539b311","author":{"email":"ca56a5f347880579ad16e22284df28d13a310849@gmail.com","name":"Tom Kaitchuck"},"message":"Merge pull request #44 from Loudr/fixes-43\n\nRaise Exception when default cloud storage pipeline is not set","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/2561501271f510c89db9dd2e9de1691eb539b311"},{"sha":"e1f5273d970c11757305939c7cbfef8ed238ba24","author":{"email":"ca56a5f347880579ad16e22284df28d13a310849@gmail.com","name":"Tom Kaitchuck"},"message":"Merge pull request #34 from vendasta/fix-testutil\n\nFix common test runner.","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/e1f5273d970c11757305939c7cbfef8ed238ba24"},{"sha":"df89325c8731647fc3e46d7ab414696bd9d024e6","author":{"email":"ca56a5f347880579ad16e22284df28d13a310849@gmail.com","name":"Tom Kaitchuck"},"message":"Merge pull request #39 from Khan/fix_tests\n\nFix testing infrastructure","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/df89325c8731647fc3e46d7ab414696bd9d024e6"},{"sha":"08693d5c4412614d779af26c9aafaaf1f895535d","author":{"email":"ca56a5f347880579ad16e22284df28d13a310849@gmail.com","name":"Tom Kaitchuck"},"message":"Merge pull request #38 from Khan/fix_zipimport\n\nAdd zipimport compatability to ResourceHandler","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/08693d5c4412614d779af26c9aafaaf1f895535d"},{"sha":"5ec60d1af30700ccdecd5069dc7ef8fce6842517","author":{"email":"5f10133efc1babfe554e2c0586806b717e1f8787@gmail.com","name":"aozarov"},"message":"ignore backend service when on MVN","distinct":true,"url":"https://api.github.com/repos/Batterii/appengine-pipelines/commits/5ec60d1af30700ccdecd5069dc7ef8fce6842517"}]}