)]}'
{"id":"openstack%2Fplacement~675648","triplet_id":"openstack%2Fplacement~master~I86e96b877945a81c10ccbb2437da4213a3d06844","project":"openstack/placement","branch":"master","hashtags":[],"change_id":"I86e96b877945a81c10ccbb2437da4213a3d06844","subject":"Use expanding bindparam in get_traits_by_provider_tree","status":"MERGED","created":"2019-08-09 15:44:43.000000000","updated":"2019-08-12 15:30:44.000000000","submitted":"2019-08-12 15:30:44.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":"675648-1565623844894-7bf71c1f","meta_rev_id":"0c5593a295a2a2d3e29af60546affc3862f485e2","_number":675648,"virtual_id_number":675648,"owner":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"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":2,"date":"2019-08-12 15:30:44.000000000","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":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"all":[{"value":2,"date":"2019-08-09 16:55:59.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":2,"date":"2019-08-12 12:38:30.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":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":"","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-12 12:38:30.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":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":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":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-09 16:55:59.000000000","updated_by":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"reviewer":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"state":"REVIEWER"},{"updated":"2019-08-12 12:38:30.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-12 15:30:44.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":"bfd2d1c70d0182b23339a82bb8c160df99f3b25c","author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"date":"2019-08-09 15:44:43.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"86b7d9638c2ee448d2a5ce18467841fcedc05613","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-08-09 16:55:59.000000000","message":"Patch Set 1: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"9963cfd4cabe355fdc996d0695692e96183bead3","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-09 23:11:32.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- grenade-py3 https://logs.opendev.org/48/675648/1/check/grenade-py3/673b465/ : SUCCESS in 1h 03m 06s\n- tempest-integrated-placement https://logs.opendev.org/48/675648/1/check/tempest-integrated-placement/31153b4/ : SUCCESS in 1h 10m 53s\n- openstack-tox-docs https://logs.opendev.org/48/675648/1/check/openstack-tox-docs/0adf23d/html/ : SUCCESS in 4m 35s\n- openstack-tox-cover https://logs.opendev.org/48/675648/1/check/openstack-tox-cover/8dd8164/cover/ : SUCCESS in 6m 20s\n- openstack-tox-lower-constraints https://logs.opendev.org/48/675648/1/check/openstack-tox-lower-constraints/d12ff44/ : SUCCESS in 5m 16s\n- openstack-tox-pep8 https://logs.opendev.org/48/675648/1/check/openstack-tox-pep8/16b9d98/ : SUCCESS in 6m 03s\n- openstack-tox-py27 https://logs.opendev.org/48/675648/1/check/openstack-tox-py27/aba600e/ : SUCCESS in 5m 47s\n- openstack-tox-py36 https://logs.opendev.org/48/675648/1/check/openstack-tox-py36/366d5ef/ : SUCCESS in 3m 57s\n- openstack-tox-py37 https://logs.opendev.org/48/675648/1/check/openstack-tox-py37/26cd1bb/ : SUCCESS in 4m 16s\n- openstack-tox-functional https://logs.opendev.org/48/675648/1/check/openstack-tox-functional/2159abd/ : SUCCESS in 4m 43s\n- openstack-tox-functional-py36 https://logs.opendev.org/48/675648/1/check/openstack-tox-functional-py36/376fdb8/ : SUCCESS in 4m 02s\n- openstack-tox-functional-py37 https://logs.opendev.org/48/675648/1/check/openstack-tox-functional-py37/aa0fc25/ : SUCCESS in 4m 38s\n- placement-nova-tox-functional-py37 https://logs.opendev.org/48/675648/1/check/placement-nova-tox-functional-py37/8abe28e/ : SUCCESS in 11m 10s\n- placement-nested-perfload https://logs.opendev.org/48/675648/1/check/placement-nested-perfload/9efaa4a/ : SUCCESS in 35m 27s (non-voting)\n- placement-perfload https://logs.opendev.org/48/675648/1/check/placement-perfload/449c728/ : SUCCESS in 12m 33s (non-voting)\n- tempest-ipv6-only https://logs.opendev.org/48/675648/1/check/tempest-ipv6-only/8f3d6b9/ : SUCCESS in 1h 04m 08s","accounts_in_message":[],"_revision_number":1},{"id":"bc7f8f126947c3b233772d2765c5d6cf467fc3d1","author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"date":"2019-08-12 12:38:30.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1\n\nThis can go as others [1][2][3] did.\n\n[1] https://review.opendev.org/#/c/674048\n[2] https://review.opendev.org/#/c/674647\n[3] https://review.opendev.org/#/c/675287","accounts_in_message":[],"_revision_number":1},{"id":"07a443ac97f86de1502a606d6ea973efde8fcac1","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-12 12:38:52.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"2869175500325df638755890ab38ad6ae9e11a6f","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-12 15:30:44.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- grenade-py3 https://logs.opendev.org/48/675648/1/gate/grenade-py3/be4aebb/ : SUCCESS in 1h 03m 24s\n- tempest-integrated-placement https://logs.opendev.org/48/675648/1/gate/tempest-integrated-placement/7b6fc8a/ : SUCCESS in 1h 17m 17s\n- openstack-tox-docs https://logs.opendev.org/48/675648/1/gate/openstack-tox-docs/5238e40/html/ : SUCCESS in 3m 47s\n- openstack-tox-lower-constraints https://logs.opendev.org/48/675648/1/gate/openstack-tox-lower-constraints/32ba606/ : SUCCESS in 4m 49s\n- openstack-tox-pep8 https://logs.opendev.org/48/675648/1/gate/openstack-tox-pep8/9ac1ad4/ : SUCCESS in 4m 39s\n- openstack-tox-py27 https://logs.opendev.org/48/675648/1/gate/openstack-tox-py27/d5d9baa/ : SUCCESS in 4m 06s\n- openstack-tox-py36 https://logs.opendev.org/48/675648/1/gate/openstack-tox-py36/55468b8/ : SUCCESS in 4m 06s\n- openstack-tox-py37 https://logs.opendev.org/48/675648/1/gate/openstack-tox-py37/6806e23/ : SUCCESS in 4m 45s\n- openstack-tox-functional https://logs.opendev.org/48/675648/1/gate/openstack-tox-functional/6bd1dcb/ : SUCCESS in 5m 23s\n- openstack-tox-functional-py36 https://logs.opendev.org/48/675648/1/gate/openstack-tox-functional-py36/05109d3/ : SUCCESS in 6m 20s\n- openstack-tox-functional-py37 https://logs.opendev.org/48/675648/1/gate/openstack-tox-functional-py37/e247fcd/ : SUCCESS in 6m 04s\n- placement-nova-tox-functional-py37 https://logs.opendev.org/48/675648/1/gate/placement-nova-tox-functional-py37/4ade1f0/ : SUCCESS in 13m 02s\n- tempest-ipv6-only https://logs.opendev.org/48/675648/1/gate/tempest-ipv6-only/dfc3061/ : SUCCESS in 1h 08m 42s","accounts_in_message":[],"_revision_number":1},{"id":"0c5593a295a2a2d3e29af60546affc3862f485e2","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-12 15:30:44.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"3c1021817de16faefdd325f692936479ba957631","revisions":{"3c1021817de16faefdd325f692936479ba957631":{"kind":"REWORK","_number":1,"created":"2019-08-09 15:44:43.000000000","uploader":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"ref":"refs/changes/48/675648/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/placement","ref":"refs/changes/48/675648/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/placement refs/changes/48/675648/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/placement refs/changes/48/675648/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/placement refs/changes/48/675648/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/placement refs/changes/48/675648/1"}}},"commit":{"parents":[{"commit":"a86f727676f457a2f54dfab1a6c90c57130ffead","subject":"Merge \"Add __copy__ method to AllocationRequest{,Resource}\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/a86f727676f457a2f54dfab1a6c90c57130ffead"}]}],"author":{"name":"Chris Dent","email":"cdent@anticdent.org","date":"2019-08-09 15:43:32.000000000","tz":60},"committer":{"name":"Chris Dent","email":"cdent@anticdent.org","date":"2019-08-09 15:43:32.000000000","tz":60},"subject":"Use expanding bindparam in get_traits_by_provider_tree","message":"Use expanding bindparam in get_traits_by_provider_tree\n\nWhen there are many root_ids, the in_ in get_traits_by_provider_tree\ncan be expensive (according to profiling data), so we switch it\nhere to an expanding bindparam which has proven elsewher to help.\n\nChange-Id: I86e96b877945a81c10ccbb2437da4213a3d06844\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/3c1021817de16faefdd325f692936479ba957631"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/placement/commit/3c1021817de16faefdd325f692936479ba957631"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}