)]}'
{"id":"openstack%2Fplacement~674349","triplet_id":"openstack%2Fplacement~master~I8120b30271ecdaf7f557a965102d03f67b6fce85","project":"openstack/placement","branch":"master","topic":"optimize-_build_provider_summaries","hashtags":[],"change_id":"I8120b30271ecdaf7f557a965102d03f67b6fce85","subject":"Further optimize _build_provider_summaries","status":"MERGED","created":"2019-08-02 16:34:37.000000000","updated":"2019-08-07 09:45:08.000000000","submitted":"2019-08-07 09:45:08.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"674349-1565171108452-01c9335f","meta_rev_id":"03f0b85d6c391bec61f0819d1a8c23e96e032fac","_number":674349,"virtual_id_number":674349,"owner":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":2,"date":"2019-08-07 09:45:08.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"recommended":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"all":[{"value":1,"date":"2019-08-06 08:33:13.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":2,"date":"2019-08-07 08:01:03.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":2,"date":"2019-08-06 08:33:13.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"all":[{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":1,"date":"2019-08-07 08:01:03.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},{"value":0,"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-08-05 14:40:49.000000000","updated_by":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"reviewer":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"state":"REVIEWER"},{"updated":"2019-08-07 08:01:03.000000000","updated_by":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"reviewer":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"state":"REVIEWER"},{"updated":"2019-08-07 09:45:08.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"618cb1473d8a47d55e1906023c8d795337afe3b6","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-08-02 16:34:37.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a0ba2f493d3e59c71cb1b0a7778ffbf445eef0b7","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-08-02 16:35:42.000000000","message":"Patch Set 1:\n\nI didn\u0027t profile anything. I\u0027d be interested to know if this helps.","accounts_in_message":[],"_revision_number":1},{"id":"1eda1fff1fc56b81b4391a8c7b7be55ef37d6cc1","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-02 17:57:53.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade-py3 https://logs.opendev.org/49/674349/1/check/grenade-py3/ab58226/ : SUCCESS in 1h 01m 19s\n- tempest-integrated-placement https://logs.opendev.org/49/674349/1/check/tempest-integrated-placement/d4fc1a5/ : SUCCESS in 1h 17m 27s\n- openstack-tox-docs https://logs.opendev.org/49/674349/1/check/openstack-tox-docs/f7575b7/html/ : SUCCESS in 4m 21s\n- openstack-tox-cover https://logs.opendev.org/49/674349/1/check/openstack-tox-cover/3856144/cover/ : SUCCESS in 7m 00s\n- openstack-tox-lower-constraints https://logs.opendev.org/49/674349/1/check/openstack-tox-lower-constraints/6b53b90/ : SUCCESS in 5m 15s\n- openstack-tox-pep8 https://logs.opendev.org/49/674349/1/check/openstack-tox-pep8/627da3a/ : SUCCESS in 7m 36s\n- openstack-tox-py27 https://logs.opendev.org/49/674349/1/check/openstack-tox-py27/a75e331/ : SUCCESS in 6m 48s\n- openstack-tox-py36 https://logs.opendev.org/49/674349/1/check/openstack-tox-py36/c619972/ : SUCCESS in 7m 26s\n- openstack-tox-py37 https://logs.opendev.org/49/674349/1/check/openstack-tox-py37/6dda79c/ : SUCCESS in 4m 39s\n- openstack-tox-functional https://logs.opendev.org/49/674349/1/check/openstack-tox-functional/8ed99cd/ : SUCCESS in 5m 39s\n- openstack-tox-functional-py36 https://logs.opendev.org/49/674349/1/check/openstack-tox-functional-py36/1674c6a/ : SUCCESS in 7m 50s\n- openstack-tox-functional-py37 https://logs.opendev.org/49/674349/1/check/openstack-tox-functional-py37/107acf1/ : SUCCESS in 9m 10s\n- placement-nova-tox-functional-py37 https://logs.opendev.org/49/674349/1/check/placement-nova-tox-functional-py37/c0e2e0c/ : SUCCESS in 17m 19s\n- placement-nested-perfload https://logs.opendev.org/49/674349/1/check/placement-nested-perfload/5d0705d/ : SUCCESS in 19m 21s (non-voting)\n- placement-perfload https://logs.opendev.org/49/674349/1/check/placement-perfload/1549e9f/ : SUCCESS in 8m 18s (non-voting)\n- tempest-ipv6-only https://logs.opendev.org/49/674349/1/check/tempest-ipv6-only/80c1eb5/ : SUCCESS in 1h 04m 58s","accounts_in_message":[],"_revision_number":1},{"id":"82e2109de11fbc7fc46c5fa0a61c8e2177132ef7","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-08-05 10:58:55.000000000","message":"Patch Set 1:\n\ni\u0027m putting this in the profiling machine\n\nbut initial reads suggest it ought to be good\n\nmore in a mo","accounts_in_message":[],"_revision_number":1},{"id":"66970f6a2c49c56355815f656d1590bb3fc69d6e","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-08-05 11:11:38.000000000","message":"Patch Set 1: Code-Review+2\n\nSince we\u0027ve split up the stack, it\u0027s hard to see in the profiling data the impact of this because items from some of the other change are showing up too much. However, comparing it with the perfload data and the profile of the previous patch it\u0027s a win.\n\nOne thing that becomes clear from this is that we get usage info for the root_ids in every RequestGroup, there\u0027s presumably some overlap there?","accounts_in_message":[],"_revision_number":1},{"id":"6587251409a1875888e6b4ea0694644a1f66b96d","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-08-05 14:01:02.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.","accounts_in_message":[],"_revision_number":2},{"id":"10d31cf62f089fdfa5ccfdf6cf36ff8c4c438515","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-08-05 14:33:26.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"707779f252638b94af8a0690abdd1ef8a6e658ab","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-08-05 14:35:49.000000000","message":"Patch Set 3:\n\nps3 adds a return if there are no rp_ids available to process after we filter\n\nThis avoids a  \"\u0027expanding\u0027 parameters with an empty list not supported until SQLAlchemy 1.3\" error when using sqlalchemy 1.2\n\nThat we were using 1.2 helped us to discover some code to skip...","accounts_in_message":[],"_revision_number":3},{"id":"9cc1ad29eaa9ab3bfe68db06fe50695f27b82845","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-08-05 14:39:14.000000000","message":"Patch Set 3: Code-Review+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"f4484d28e8245347d5c575713ff85003e6509541","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-08-05 14:40:49.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"c7779805206e9927c236b0eeaec10153f871ba4d","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-05 16:04:31.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade-py3 https://logs.opendev.org/49/674349/3/check/grenade-py3/5fcfc45/ : SUCCESS in 1h 00m 02s\n- tempest-integrated-placement https://logs.opendev.org/49/674349/3/check/tempest-integrated-placement/f560030/ : SUCCESS in 1h 28m 30s\n- openstack-tox-docs https://logs.opendev.org/49/674349/3/check/openstack-tox-docs/359b5e2/html/ : SUCCESS in 4m 12s\n- openstack-tox-cover https://logs.opendev.org/49/674349/3/check/openstack-tox-cover/f02674d/cover/ : SUCCESS in 7m 32s\n- openstack-tox-lower-constraints https://logs.opendev.org/49/674349/3/check/openstack-tox-lower-constraints/7cc4278/ : SUCCESS in 6m 04s\n- openstack-tox-pep8 https://logs.opendev.org/49/674349/3/check/openstack-tox-pep8/1b08ce8/ : SUCCESS in 4m 09s\n- openstack-tox-py27 https://logs.opendev.org/49/674349/3/check/openstack-tox-py27/c697f5b/ : SUCCESS in 4m 21s\n- openstack-tox-py36 https://logs.opendev.org/49/674349/3/check/openstack-tox-py36/ffc12ea/ : SUCCESS in 5m 55s\n- openstack-tox-py37 https://logs.opendev.org/49/674349/3/check/openstack-tox-py37/1f88066/ : SUCCESS in 6m 10s\n- openstack-tox-functional https://logs.opendev.org/49/674349/3/check/openstack-tox-functional/735ddaf/ : SUCCESS in 5m 58s\n- openstack-tox-functional-py36 https://logs.opendev.org/49/674349/3/check/openstack-tox-functional-py36/3a27c38/ : SUCCESS in 7m 25s\n- openstack-tox-functional-py37 https://logs.opendev.org/49/674349/3/check/openstack-tox-functional-py37/33857ca/ : SUCCESS in 7m 32s\n- placement-nova-tox-functional-py37 https://logs.opendev.org/49/674349/3/check/placement-nova-tox-functional-py37/ad9b2f1/ : SUCCESS in 11m 36s\n- placement-nested-perfload https://logs.opendev.org/49/674349/3/check/placement-nested-perfload/46ec1e6/ : SUCCESS in 19m 06s (non-voting)\n- placement-perfload https://logs.opendev.org/49/674349/3/check/placement-perfload/4cb3366/ : SUCCESS in 6m 57s (non-voting)\n- tempest-ipv6-only https://logs.opendev.org/49/674349/3/check/tempest-ipv6-only/85c6931/ : SUCCESS in 1h 01m 27s","accounts_in_message":[],"_revision_number":3},{"id":"74b3262d1a7b7f5d9c6cab9e215dd7cf1d8cfcfb","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-08-06 08:33:13.000000000","message":"Uploaded patch set 4: Patch Set 3 was rebased.","accounts_in_message":[],"_revision_number":4},{"id":"c2fa329ed350f3acbcd3e7d9cddbd88411b07c98","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-06 11:57:23.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade-py3 https://logs.opendev.org/49/674349/4/check/grenade-py3/3e94014/ : SUCCESS in 55m 38s\n- tempest-integrated-placement https://logs.opendev.org/49/674349/4/check/tempest-integrated-placement/6622215/ : SUCCESS in 1h 24m 09s\n- openstack-tox-docs https://logs.opendev.org/49/674349/4/check/openstack-tox-docs/d51dbac/html/ : SUCCESS in 5m 20s\n- openstack-tox-cover https://logs.opendev.org/49/674349/4/check/openstack-tox-cover/cefc6ff/cover/ : SUCCESS in 5m 33s\n- openstack-tox-lower-constraints https://logs.opendev.org/49/674349/4/check/openstack-tox-lower-constraints/73818a0/ : SUCCESS in 6m 06s\n- openstack-tox-pep8 https://logs.opendev.org/49/674349/4/check/openstack-tox-pep8/e5aa814/ : SUCCESS in 7m 29s\n- openstack-tox-py27 https://logs.opendev.org/49/674349/4/check/openstack-tox-py27/f8732ac/ : SUCCESS in 5m 33s\n- openstack-tox-py36 https://logs.opendev.org/49/674349/4/check/openstack-tox-py36/652fecf/ : SUCCESS in 5m 28s\n- openstack-tox-py37 https://logs.opendev.org/49/674349/4/check/openstack-tox-py37/fd7295e/ : SUCCESS in 5m 43s\n- openstack-tox-functional https://logs.opendev.org/49/674349/4/check/openstack-tox-functional/3f459ee/ : SUCCESS in 5m 49s\n- openstack-tox-functional-py36 https://logs.opendev.org/49/674349/4/check/openstack-tox-functional-py36/5a4d499/ : SUCCESS in 6m 05s\n- openstack-tox-functional-py37 https://logs.opendev.org/49/674349/4/check/openstack-tox-functional-py37/ba0c66d/ : SUCCESS in 6m 13s\n- placement-nova-tox-functional-py37 https://logs.opendev.org/49/674349/4/check/placement-nova-tox-functional-py37/1ec9dda/ : SUCCESS in 13m 36s\n- placement-nested-perfload https://logs.opendev.org/49/674349/4/check/placement-nested-perfload/9388ee3/ : SUCCESS in 17m 58s (non-voting)\n- placement-perfload https://logs.opendev.org/49/674349/4/check/placement-perfload/8152162/ : SUCCESS in 7m 38s (non-voting)\n- tempest-ipv6-only https://logs.opendev.org/49/674349/4/check/tempest-ipv6-only/a683199/ : SUCCESS in 1h 04m 05s","accounts_in_message":[],"_revision_number":4},{"id":"a7852795a00f4b334fe97ebb0f9cd5262c03557e","author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"date":"2019-08-07 08:01:03.000000000","message":"Patch Set 4: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":4},{"id":"3c81aad06e59026ecdb92188025b898bc11ace13","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-07 08:01:17.000000000","message":"Patch Set 4: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":4},{"id":"b2de216debddeb62c921b1966fcbd10544294bfa","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-07 09:45:08.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":4},{"id":"1c2f654c9c282d843f2e74b0ac6da5a185897163","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-07 09:45:08.000000000","message":"Patch Set 4: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- grenade-py3 https://logs.opendev.org/49/674349/4/gate/grenade-py3/df853e9/ : SUCCESS in 1h 03m 05s\n- tempest-integrated-placement https://logs.opendev.org/49/674349/4/gate/tempest-integrated-placement/eb99cd6/ : SUCCESS in 1h 29m 24s\n- openstack-tox-docs https://logs.opendev.org/49/674349/4/gate/openstack-tox-docs/c82bb63/html/ : SUCCESS in 7m 21s\n- openstack-tox-lower-constraints https://logs.opendev.org/49/674349/4/gate/openstack-tox-lower-constraints/c6773c6/ : SUCCESS in 7m 07s\n- openstack-tox-pep8 https://logs.opendev.org/49/674349/4/gate/openstack-tox-pep8/77bff8a/ : SUCCESS in 6m 40s\n- openstack-tox-py27 https://logs.opendev.org/49/674349/4/gate/openstack-tox-py27/7c184cc/ : SUCCESS in 5m 37s\n- openstack-tox-py36 https://logs.opendev.org/49/674349/4/gate/openstack-tox-py36/5664aad/ : SUCCESS in 5m 33s\n- openstack-tox-py37 https://logs.opendev.org/49/674349/4/gate/openstack-tox-py37/0d7dfab/ : SUCCESS in 8m 30s\n- openstack-tox-functional https://logs.opendev.org/49/674349/4/gate/openstack-tox-functional/5a8bd9b/ : SUCCESS in 6m 32s\n- openstack-tox-functional-py36 https://logs.opendev.org/49/674349/4/gate/openstack-tox-functional-py36/74332cc/ : SUCCESS in 5m 30s\n- openstack-tox-functional-py37 https://logs.opendev.org/49/674349/4/gate/openstack-tox-functional-py37/9b1f491/ : SUCCESS in 5m 22s\n- placement-nova-tox-functional-py37 https://logs.opendev.org/49/674349/4/gate/placement-nova-tox-functional-py37/3dd70bd/ : SUCCESS in 16m 35s\n- tempest-ipv6-only https://logs.opendev.org/49/674349/4/gate/tempest-ipv6-only/720b62c/ : SUCCESS in 1h 15m 34s","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"0386f5340787091970ebb0c4b96535635a7f405b","revisions":{"70a0fbf4e66d34865716a38e10cb234a5a819121":{"kind":"REWORK","_number":1,"created":"2019-08-02 16:34:37.000000000","uploader":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"ref":"refs/changes/49/674349/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/49/674349/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/49/674349/1"}}},"commit":{"parents":[{"commit":"154a46a147a4848557877a51b9deca2241c7c9b4","subject":"Add RequestWideSearchContext.summaries_by_id","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/154a46a147a4848557877a51b9deca2241c7c9b4"}]}],"author":{"name":"Eric Fried","email":"openstack@fried.cc","date":"2019-08-02 16:32:54.000000000","tz":-300},"committer":{"name":"Eric Fried","email":"openstack@fried.cc","date":"2019-08-02 16:32:54.000000000","tz":-300},"subject":"Further optimize _build_provider_summaries","message":"Further optimize _build_provider_summaries\n\nSince usages don\u0027t change per operation, if we enter\n_build_provider_summaries having already created a summary for a given\nresource provider, we can skip processing all the usages for that\nresource provider.\n\nChange-Id: I8120b30271ecdaf7f557a965102d03f67b6fce85\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/70a0fbf4e66d34865716a38e10cb234a5a819121"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/70a0fbf4e66d34865716a38e10cb234a5a819121"}]},"branch":"refs/heads/master"},"4d6b0418ad1361b2db2173e90802043afce22823":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2019-08-05 14:01:02.000000000","uploader":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"ref":"refs/changes/49/674349/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/49/674349/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/49/674349/2"}}},"commit":{"parents":[{"commit":"dff83d155bf909bf5814ebd5522173a6878f7b92","subject":"Add RequestWideSearchContext.summaries_by_id","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/dff83d155bf909bf5814ebd5522173a6878f7b92"}]}],"author":{"name":"Eric Fried","email":"openstack@fried.cc","date":"2019-08-02 16:32:54.000000000","tz":-300},"committer":{"name":"Chris Dent","email":"cdent@anticdent.org","date":"2019-08-05 14:00:18.000000000","tz":60},"subject":"Further optimize _build_provider_summaries","message":"Further optimize _build_provider_summaries\n\nSince usages don\u0027t change per operation, if we enter\n_build_provider_summaries having already created a summary for a given\nresource provider, we can skip processing all the usages for that\nresource provider.\n\nChange-Id: I8120b30271ecdaf7f557a965102d03f67b6fce85\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/4d6b0418ad1361b2db2173e90802043afce22823"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/4d6b0418ad1361b2db2173e90802043afce22823"}]},"branch":"refs/heads/master"},"e2e128c505124370b096ec4f8444cbba30fb0f41":{"kind":"REWORK","_number":3,"created":"2019-08-05 14:33:26.000000000","uploader":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"ref":"refs/changes/49/674349/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/49/674349/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/49/674349/3"}}},"commit":{"parents":[{"commit":"dff83d155bf909bf5814ebd5522173a6878f7b92","subject":"Add RequestWideSearchContext.summaries_by_id","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/dff83d155bf909bf5814ebd5522173a6878f7b92"}]}],"author":{"name":"Eric Fried","email":"openstack@fried.cc","date":"2019-08-02 16:32:54.000000000","tz":-300},"committer":{"name":"Chris Dent","email":"cdent@anticdent.org","date":"2019-08-05 14:31:38.000000000","tz":60},"subject":"Further optimize _build_provider_summaries","message":"Further optimize _build_provider_summaries\n\nSince usages don\u0027t change per operation, if we enter\n_build_provider_summaries having already created a summary for a given\nresource provider, we can skip processing all the usages for that\nresource provider.\n\nAnd, if we have no rp_ids left over, we can skip the method\nentirely.\n\nChange-Id: I8120b30271ecdaf7f557a965102d03f67b6fce85\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/e2e128c505124370b096ec4f8444cbba30fb0f41"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/e2e128c505124370b096ec4f8444cbba30fb0f41"}]},"branch":"refs/heads/master"},"0386f5340787091970ebb0c4b96535635a7f405b":{"kind":"TRIVIAL_REBASE","_number":4,"created":"2019-08-06 08:33:13.000000000","uploader":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"ref":"refs/changes/49/674349/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/49/674349/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/49/674349/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/49/674349/4"}}},"commit":{"parents":[{"commit":"6ef820c98272d20b1d153ab35873536c64a0e4ae","subject":"Add RequestWideSearchContext.summaries_by_id","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/6ef820c98272d20b1d153ab35873536c64a0e4ae"}]}],"author":{"name":"Eric Fried","email":"openstack@fried.cc","date":"2019-08-02 16:32:54.000000000","tz":-300},"committer":{"name":"Chris Dent","email":"cdent@anticdent.org","date":"2019-08-06 08:29:35.000000000","tz":60},"subject":"Further optimize _build_provider_summaries","message":"Further optimize _build_provider_summaries\n\nSince usages don\u0027t change per operation, if we enter\n_build_provider_summaries having already created a summary for a given\nresource provider, we can skip processing all the usages for that\nresource provider.\n\nAnd, if we have no rp_ids left over, we can skip the method\nentirely.\n\nChange-Id: I8120b30271ecdaf7f557a965102d03f67b6fce85\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/0386f5340787091970ebb0c4b96535635a7f405b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/0386f5340787091970ebb0c4b96535635a7f405b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}