{"push_id":551112578,"size":15,"distinct_size":15,"ref":"refs/heads/master","head":"c7dae7c908e34e4908a69f1aff8cbd5ddf820fac","before":"eb302661aa2a008902850b6c30c080f1a4afd8ea","commits":[{"sha":"895e8482a58aae70891491c13ab527e0ceb23a89","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Combine the id-class-by and id-by cases for doubly delegated properties","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/895e8482a58aae70891491c13ab527e0ceb23a89"},{"sha":"6416b407da1e74b810781f339c2388cede9f0eeb","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"bring up $bridging_identifiers in preparation for combining via-to section","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/6416b407da1e74b810781f339c2388cede9f0eeb"},{"sha":"5f1a42a502afaca6689a5fcfa55275a888560c81","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"combine the via case with both id-by cases for doubly delegated properties","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/5f1a42a502afaca6689a5fcfa55275a888560c81"},{"sha":"8552ec8ac0708a1ac86f404b42a9411a5dd7a273","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Rename variable\n\n$bridging_identifiers -> $bridge_linking_values\n\nsince adding the via-case, it's not always storing result ID values","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/8552ec8ac0708a1ac86f404b42a9411a5dd7a273"},{"sha":"017a01589ea37eee597c478340748851f72ff265","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Use the via-via case only when there's a single property linking the bridge to the result\n\nIf there's more than 1 ID property linking the bridge to the result, we\ncan't use this.  The simple thing to do would be to generate a list of property\nnames and supply listrefs for each property when get get(), but that's wrong.\nThat query would mean\n    where a in <list_a> and b in <list_b>\n\nbut we really want a query like\n    where (a = a1 and b = b1) or (a = a2 and b = b2)","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/017a01589ea37eee597c478340748851f72ff265"},{"sha":"5f25ef470324788acbe5010323e8ae21e00f0a23","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Don't use the double-delegated speedup for reverse-as properties\n\nreverse-as doesn't match any of the cases it checks for, so the\n$bridge_crosser ends up crashing $result_class_resolver isn't defined","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/5f25ef470324788acbe5010323e8ae21e00f0a23"},{"sha":"63f603f646647c53b607c8430dddef57e40b4efb","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Rename variable\n\nThe listref is holding property names, not values","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/63f603f646647c53b607c8430dddef57e40b4efb"},{"sha":"97fc1d4af732dc2e3b57cfaff148ae8aa0f42eed","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Add a handler for via-reverseas double delegations","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/97fc1d4af732dc2e3b57cfaff148ae8aa0f42eed"},{"sha":"58b2055b7db384ead0547d246b79d5b807b5313a","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Add a comment about what we're doing in this section","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/58b2055b7db384ead0547d246b79d5b807b5313a"},{"sha":"50c6e5521d7ee69fbf10e391254cd1c3854d8de3","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Only use this efficiency when there's a single linking property.\n\nSee the message in commit 017a01589 for the reason why","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/50c6e5521d7ee69fbf10e391254cd1c3854d8de3"},{"sha":"e1584466e02bf0fefac5ca11c8d6a721173d27e0","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Use the efficient query code only if we didn't fall through\n\nFor a couple of cases (via/via and via/reverse-as), we can only use the more\nefficient querying if there's a single foreign key property in the second\ndelegation.  Multiple FKs would cause execution to fall through without setting\nthe values needed by the efficient code.","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/e1584466e02bf0fefac5ca11c8d6a721173d27e0"},{"sha":"85785170a955f134f5b24e634aaf3ea5fdc2be83","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Extract functions\n\nextracted $make_results_sorter to _make_results_sorter_for_doubly_delegated_bridge_crosser()\nextracted if() section for doubly delegated properties to _resolve_bridge_crosser_for_doubly_delegated_property()","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/85785170a955f134f5b24e634aaf3ea5fdc2be83"},{"sha":"8225098aa2d04e6e739fcdc0b06681e1ee479897","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Add test case for doubly delegated via/via with multiple FK properties","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/8225098aa2d04e6e739fcdc0b06681e1ee479897"},{"sha":"64911c73d0107832869e7084600e76470f4f2647","author":{"email":"97107fc477a644f4fc2c182179acf1d304b302df@genome.wustl.edu","name":"Anthony Brummett"},"message":"Remove redundant else branch\n\nAfter refactoring, both branches are the same","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/64911c73d0107832869e7084600e76470f4f2647"},{"sha":"c7dae7c908e34e4908a69f1aff8cbd5ddf820fac","author":{"email":"fbe9f7a8dfd6f842583f1cdaf20c61b83355f3ad@gmail.com","name":"Anthony Brummett"},"message":"Merge pull request #53 from brummett/doubly-delegated-refactor\n\nDoubly delegated refactor","distinct":true,"url":"https://api.github.com/repos/genome/UR/commits/c7dae7c908e34e4908a69f1aff8cbd5ddf820fac"}]}