)]}'
{"id":"openstack%2Fpython-troveclient~361871","triplet_id":"openstack%2Fpython-troveclient~master~If81962c8027442f75d27cb048ab0acdd33311f8c","project":"openstack/python-troveclient","branch":"master","topic":"bug/1571548","hashtags":[],"change_id":"If81962c8027442f75d27cb048ab0acdd33311f8c","subject":"Replace functions \u0027Dict.get\u0027 and \u0027del\u0027 with \u0027Dict.pop\u0027","status":"ABANDONED","created":"2016-08-29 06:15:06.000000000","updated":"2016-12-28 03:37:34.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"68cd321bc9745082529068e66b25bd68533fb920","_number":361871,"virtual_id_number":361871,"owner":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},{"value":1,"date":"2016-10-12 11:02:21.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"all":[{"value":-1,"date":"2016-10-12 10:30:55.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"}],"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":{"all":[{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"}],"values":{"-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-08-30 22:57:14.000000000","updated_by":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"reviewer":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"state":"REVIEWER"},{"updated":"2016-10-12 10:30:55.000000000","updated_by":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"reviewer":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"state":"REVIEWER"},{"updated":"2016-10-12 11:02:21.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"019143fd18765d9182d36b36fa2a6467a665eba3","author":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"date":"2016-08-29 06:15:06.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"de11db581058701f42a68fd0d996a0c1b6c37308","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-29 07:51:58.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-troveclient-docs-ubuntu-xenial http://docs-draft.openstack.org/71/361871/1/check/gate-python-troveclient-docs-ubuntu-xenial/693951a//doc/build/html/ : SUCCESS in 3m 12s\n- gate-python-troveclient-pep8-ubuntu-xenial http://logs.openstack.org/71/361871/1/check/gate-python-troveclient-pep8-ubuntu-xenial/b8c7984/ : SUCCESS in 2m 10s\n- gate-python-troveclient-python27-ubuntu-xenial http://logs.openstack.org/71/361871/1/check/gate-python-troveclient-python27-ubuntu-xenial/81d8407/ : SUCCESS in 2m 20s\n- gate-python-troveclient-python34 http://logs.openstack.org/71/361871/1/check/gate-python-troveclient-python34/3987079/ : SUCCESS in 3m 03s\n- gate-python-troveclient-python35-nv http://logs.openstack.org/71/361871/1/check/gate-python-troveclient-python35-nv/af86f45/ : SUCCESS in 2m 45s (non-voting)\n- gate-python-troveclient-pypy http://logs.openstack.org/71/361871/1/check/gate-python-troveclient-pypy/7116675/ : FAILURE in 2m 24s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-troveclient http://logs.openstack.org/71/361871/1/check/gate-tempest-dsvm-neutron-src-python-troveclient/03d50d8/ : SUCCESS in 1h 15m 48s\n- gate-trove-functional-dsvm-mysql http://logs.openstack.org/71/361871/1/check/gate-trove-functional-dsvm-mysql/da475eb/ : FAILURE in 1h 35m 23s","accounts_in_message":[],"_revision_number":1},{"id":"9ede76d042cfc63898a66b6f703a06e33c87f2b9","author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"date":"2016-08-29 10:52:46.000000000","message":"Patch Set 1: Code-Review-1\n\nThese two are NOT the same especially in the case where json.dumps() throws an exception for some reason.\n\nIn any event, even if this is a trivial fix, it needs a bug.","accounts_in_message":[],"_revision_number":1},{"id":"7be57a23b2d563be38e029f8a58429b5ba43077b","author":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"date":"2016-08-29 13:25:53.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"d40af0ff5d4ffb829135b8a6690c2b676c82a3a2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-29 15:08:15.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-troveclient-docs-ubuntu-xenial http://docs-draft.openstack.org/71/361871/1/check/gate-python-troveclient-docs-ubuntu-xenial/7c09dd9//doc/build/html/ : SUCCESS in 3m 12s\n- gate-python-troveclient-pep8-ubuntu-xenial http://logs.openstack.org/71/361871/1/check/gate-python-troveclient-pep8-ubuntu-xenial/a746f43/ : SUCCESS in 2m 05s\n- gate-python-troveclient-python27-ubuntu-xenial http://logs.openstack.org/71/361871/1/check/gate-python-troveclient-python27-ubuntu-xenial/b5cb71b/ : SUCCESS in 2m 19s\n- gate-python-troveclient-python34 http://logs.openstack.org/71/361871/1/check/gate-python-troveclient-python34/3fe4c9c/ : SUCCESS in 5m 37s\n- gate-python-troveclient-python35-nv http://logs.openstack.org/71/361871/1/check/gate-python-troveclient-python35-nv/c506f6d/ : SUCCESS in 2m 33s (non-voting)\n- gate-python-troveclient-pypy http://logs.openstack.org/71/361871/1/check/gate-python-troveclient-pypy/314b972/ : FAILURE in 2m 15s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-troveclient http://logs.openstack.org/71/361871/1/check/gate-tempest-dsvm-neutron-src-python-troveclient/9a74606/ : SUCCESS in 1h 07m 15s\n- gate-trove-functional-dsvm-mysql http://logs.openstack.org/71/361871/1/check/gate-trove-functional-dsvm-mysql/89aacb3/ : SUCCESS in 1h 40m 25s","accounts_in_message":[],"_revision_number":1},{"id":"7a8366ceab60ac3c988206ae4e8adb4de269fc74","author":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"date":"2016-08-30 10:39:17.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"da947e115aea4a309cef9d20416481ebacedf0dd","author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"date":"2016-08-30 11:21:37.000000000","message":"Patch Set 2: Code-Review-1\n\nno, popping the value before calling json.dumps() doesn\u0027t make it any different. if json.dumps() throws an exception then the thing remains popped.\n\nThe bug you referenced is very clearly cinder specific ...\n\nPersonally, I see no value in this change but that\u0027s just me.","accounts_in_message":[],"_revision_number":2},{"id":"b9ca89f13f40f097405dc8d73b5a06172c9a3704","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-30 12:06:47.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-troveclient-docs-ubuntu-xenial http://docs-draft.openstack.org/71/361871/2/check/gate-python-troveclient-docs-ubuntu-xenial/cc297b0//doc/build/html/ : SUCCESS in 3m 18s\n- gate-python-troveclient-pep8-ubuntu-xenial http://logs.openstack.org/71/361871/2/check/gate-python-troveclient-pep8-ubuntu-xenial/42dd116/ : SUCCESS in 4m 31s\n- gate-python-troveclient-python27-ubuntu-xenial http://logs.openstack.org/71/361871/2/check/gate-python-troveclient-python27-ubuntu-xenial/c65e627/ : SUCCESS in 2m 14s\n- gate-python-troveclient-python34 http://logs.openstack.org/71/361871/2/check/gate-python-troveclient-python34/2d0512e/ : SUCCESS in 2m 27s\n- gate-python-troveclient-python35-nv http://logs.openstack.org/71/361871/2/check/gate-python-troveclient-python35-nv/96adb51/ : SUCCESS in 3m 28s (non-voting)\n- gate-python-troveclient-pypy http://logs.openstack.org/71/361871/2/check/gate-python-troveclient-pypy/200471a/ : FAILURE in 3m 43s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-troveclient http://logs.openstack.org/71/361871/2/check/gate-tempest-dsvm-neutron-src-python-troveclient/eb46cde/ : SUCCESS in 1h 02m 07s\n- gate-trove-functional-dsvm-mysql http://logs.openstack.org/71/361871/2/check/gate-trove-functional-dsvm-mysql/b9ac7a8/ : SUCCESS in 1h 26m 24s","accounts_in_message":[],"_revision_number":2},{"id":"1cc8730fac1f916a6ed23044b484d8835bc2f845","author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"date":"2016-08-30 22:57:14.000000000","message":"Patch Set 2: Code-Review-1\n\nSo first thing is the bug was marked as invalid. This kind of fix is something you can mark as Trivial fix and push it through that way. Amrith is right about losing the value if json.dumps throws an exception. With this patch we\u0027re only losing value because now we can\u0027t print the kwargs[\u0027body\u0027] that was passed into the request function and that kind of information can be helpful in logs and error messages in the long term.","accounts_in_message":[],"_revision_number":2},{"id":"6fb32a55d0531c9107436135adcfcc2df966c0eb","author":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"date":"2016-09-07 06:44:13.000000000","message":"Patch Set 2:\n\nHi amrith and Trvor, thaks for review.\n\nYeah, point mentioned in above comment is valid but if there is any exception raised from json.dumps() then in that case these should be handle in troveclient.client.HTTPClient.request() method itself and log the \u0027body\u0027 and then raise exception from here.\nAt this point, I have confirm that the \u0027body\u0027 is not used any where so there is no data loss at all.","accounts_in_message":[],"_revision_number":2},{"id":"0d8cfc2d3a1b43c72781ec2b3f36243480472a21","author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"date":"2016-09-09 19:06:24.000000000","message":"Patch Set 2:\n\nwaiting for ocata","accounts_in_message":[],"_revision_number":2},{"id":"7bc1644a01b6c45a79e5a312424fd5a84b119246","author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"date":"2016-09-09 19:11:04.000000000","message":"Patch Set 2: Code-Review-2","accounts_in_message":[],"_revision_number":2},{"id":"c4c3da9f0862d3bc85d5c1c3ddfc4eff710794a2","author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"date":"2016-09-15 16:37:56.000000000","message":"Patch Set 2: Code-Review-1","accounts_in_message":[],"_revision_number":2},{"id":"e6b08a8b8b4b5a757f0a610317804be150c29e74","author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"date":"2016-09-23 03:24:54.000000000","message":"Patch Set 2: -Code-Review\n\nthis change appears to make no material improvement to trove, but it isn\u0027t wrong either (except in the face of exceptions). I don\u0027t think it is a worthwhile change but I\u0027ll let others decide.","accounts_in_message":[],"_revision_number":2},{"id":"c7458153ebf7caba70509e3c227f527db35f52b4","author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"date":"2016-10-11 17:22:10.000000000","message":"Patch Set 2: Code-Review-2\n\ndid not realize that this was attempting to change oslo-incubator code; can\u0027t do that. the code is obsolete anyway","accounts_in_message":[],"_revision_number":2},{"id":"0330ba04dcd420fb3d6d6cb161182e2c2c5596cb","author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"date":"2016-10-11 17:22:18.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":2},{"id":"8bb052c25ad80b1dd8beaeca3e7f16ce3f43f556","author":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"date":"2016-10-12 09:59:38.000000000","message":"Restored\n\nThe chenge in troveclient/client.py module is required because it is actually used in functionality but the changes from module troveclient/openstack/common/apiclient/client.py is used only for the test purpose so IMO this patch is good for opinion.","accounts_in_message":[],"_revision_number":2},{"id":"397f523ec08fca07761e76ea0cb6a58726052148","author":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"date":"2016-10-12 10:03:04.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"2401964c83faa7dc85a70ba3153d06cf77a2c964","author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"date":"2016-10-12 10:30:55.000000000","message":"Patch Set 3: Code-Review-1\n\nthe code does not modify oslo incubator code any longer","accounts_in_message":[],"_revision_number":3},{"id":"befd894e398c6681b75a7ed315b026bb6ecc1c2a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-10-12 11:02:21.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-troveclient-docs-ubuntu-xenial http://docs-draft.openstack.org/71/361871/3/check/gate-python-troveclient-docs-ubuntu-xenial/a4a8c52//doc/build/html/ : SUCCESS in 3m 00s\n- gate-python-troveclient-pep8-ubuntu-xenial http://logs.openstack.org/71/361871/3/check/gate-python-troveclient-pep8-ubuntu-xenial/364617c/ : SUCCESS in 3m 00s\n- gate-python-troveclient-python27-ubuntu-xenial http://logs.openstack.org/71/361871/3/check/gate-python-troveclient-python27-ubuntu-xenial/6ef76de/ : SUCCESS in 2m 41s\n- gate-python-troveclient-python34 http://logs.openstack.org/71/361871/3/check/gate-python-troveclient-python34/4e85386/ : SUCCESS in 3m 25s\n- gate-python-troveclient-python35-nv http://logs.openstack.org/71/361871/3/check/gate-python-troveclient-python35-nv/5ba602a/ : SUCCESS in 4m 33s (non-voting)\n- gate-python-troveclient-pypy http://logs.openstack.org/71/361871/3/check/gate-python-troveclient-pypy/ee22579/ : FAILURE in 2m 19s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-troveclient http://logs.openstack.org/71/361871/3/check/gate-tempest-dsvm-neutron-src-python-troveclient/bdff47d/ : SUCCESS in 42m 26s\n- gate-trove-functional-dsvm-mysql http://logs.openstack.org/71/361871/3/check/gate-trove-functional-dsvm-mysql/6a123f6/ : SUCCESS in 59m 01s","accounts_in_message":[],"_revision_number":3},{"id":"7890756968008a2cb53722884d505f575bf1cb9a","author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"date":"2016-12-28 03:37:34.000000000","message":"Patch Set 3:\n\n\"abandoned for inactivity, over 3 weeks old, has a negative review comment and is not currently marked as a WIP or WF-1. please restore the change if you wish to work on it.\"","accounts_in_message":[],"_revision_number":3},{"id":"9ca833ebb7c36fa44fff4bc45f58e23d2fb6b74f","author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"date":"2016-12-28 03:37:34.000000000","message":"Abandoned\n\n\"abandoned for inactivity, over 3 weeks old, has a negative review comment and is not currently marked as a WIP or WF-1. please restore the change if you wish to work on it.\"","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"66f19b894ef1dc86dde26a40a3cee3b911eeb93e","revisions":{"9bc2cbd3efed9659b61bd854be717085d1eca606":{"kind":"REWORK","_number":1,"created":"2016-08-29 06:15:06.000000000","uploader":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"ref":"refs/changes/71/361871/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-troveclient","ref":"refs/changes/71/361871/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/1"}}},"commit":{"parents":[{"commit":"f5616b7d5251200df753d7aff60185587ed1aada","subject":"Expose Quota.update API","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/f5616b7d5251200df753d7aff60185587ed1aada"}]}],"author":{"name":"bhagyashris","email":"bhagyashri.shewale@nttdata.com","date":"2016-08-26 09:52:21.000000000","tz":330},"committer":{"name":"bhagyashris","email":"bhagyashri.shewale@nttdata.com","date":"2016-08-26 09:57:06.000000000","tz":330},"subject":"Replace functions \u0027Dict.get\u0027 and \u0027del\u0027 with \u0027Dict.pop\u0027","message":"Replace functions \u0027Dict.get\u0027 and \u0027del\u0027 with \u0027Dict.pop\u0027\n\nRefactoring code: Making \u0027kwargs\u0027 to use single instruction: pop()\nrather than two instructions: get() and del, giving the codes a\nformat that carries through.\n\nTrivialFix\n\nChange-Id: If81962c8027442f75d27cb048ab0acdd33311f8c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/9bc2cbd3efed9659b61bd854be717085d1eca606"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/9bc2cbd3efed9659b61bd854be717085d1eca606"}]},"branch":"refs/heads/master"},"2815c66e50837ff8c582f9eb03b752fb20ac800f":{"kind":"REWORK","_number":2,"created":"2016-08-30 10:39:17.000000000","uploader":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"ref":"refs/changes/71/361871/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-troveclient","ref":"refs/changes/71/361871/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/2"}}},"commit":{"parents":[{"commit":"f5616b7d5251200df753d7aff60185587ed1aada","subject":"Expose Quota.update API","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/f5616b7d5251200df753d7aff60185587ed1aada"}]}],"author":{"name":"bhagyashris","email":"bhagyashri.shewale@nttdata.com","date":"2016-08-26 09:52:21.000000000","tz":330},"committer":{"name":"bhagyashris","email":"bhagyashri.shewale@nttdata.com","date":"2016-08-30 10:16:34.000000000","tz":330},"subject":"Replace functions \u0027Dict.get\u0027 and \u0027del\u0027 with \u0027Dict.pop\u0027","message":"Replace functions \u0027Dict.get\u0027 and \u0027del\u0027 with \u0027Dict.pop\u0027\n\nRefactoring code: Making \u0027kwargs\u0027 to use single instruction: pop()\nrather than two instructions: get() and del, giving the codes a\nformat that carries through.\n\nCloses-Bug: #1571548\nChange-Id: If81962c8027442f75d27cb048ab0acdd33311f8c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/2815c66e50837ff8c582f9eb03b752fb20ac800f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/2815c66e50837ff8c582f9eb03b752fb20ac800f"}]},"branch":"refs/heads/master"},"66f19b894ef1dc86dde26a40a3cee3b911eeb93e":{"kind":"REWORK","_number":3,"created":"2016-10-12 10:03:04.000000000","uploader":{"_account_id":20182,"name":"Bhagyashri Shewale","email":"bshewale@redhat.com","username":"bhagyashris"},"ref":"refs/changes/71/361871/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-troveclient","ref":"refs/changes/71/361871/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-troveclient refs/changes/71/361871/3"}}},"commit":{"parents":[{"commit":"ac88477948880cf4a64fbd55e26758aa35d06fc6","subject":"Merge \"Avoid use xx\u003d[]  for parameter initial value\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/ac88477948880cf4a64fbd55e26758aa35d06fc6"}]}],"author":{"name":"bhagyashris","email":"bhagyashri.shewale@nttdata.com","date":"2016-08-26 09:52:21.000000000","tz":330},"committer":{"name":"bhagyashris","email":"bhagyashri.shewale@nttdata.com","date":"2016-10-12 08:56:24.000000000","tz":330},"subject":"Replace functions \u0027Dict.get\u0027 and \u0027del\u0027 with \u0027Dict.pop\u0027","message":"Replace functions \u0027Dict.get\u0027 and \u0027del\u0027 with \u0027Dict.pop\u0027\n\nRefactoring code: Making \u0027kwargs\u0027 to use single instruction: pop()\nrather than two instructions: get() and del, giving the codes a\nformat that carries through.\n\nCloses-Bug: #1571548\nChange-Id: If81962c8027442f75d27cb048ab0acdd33311f8c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/66f19b894ef1dc86dde26a40a3cee3b911eeb93e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-troveclient/commit/66f19b894ef1dc86dde26a40a3cee3b911eeb93e"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
