Replace six.iteritems() with .items()

Registered by lihaijing on 2017-03-14

In Openstack Compute(Nova) project, six.iteritems() statement is already replaced by .item().
The related Nova bp: https://blueprints.launchpad.net/nova/+spec/replace-iteritems-with-items
The main contents of that bp are as follows:
1.As mentioned in [1], we should avoid using six.iteritems to achieve iterators. We can use dict.items instead, as it will return iterators in PY3 as well. And dict.items/keys will be more readable.

2.In py2, the performance about list should be negligible, see the link [2].

[1] https://wiki.openstack.org/wiki/Python3
[2] http://lists.openstack.org/pipermail/openstack-dev/2015-June/066391.html

Nova as one of the three major components, the replace-iteritems-with-items is a weather vane of other peojects.
So I register this BP in launchpad. Because the code affected in openstackclient is huge, so it may be reasonable to split the code changes into several patches.

Blueprint information

Status:
Not started
Approver:
None
Priority:
Undefined
Drafter:
lihaijing
Direction:
Needs approval
Assignee:
lihaijing
Definition:
New
Series goal:
None
Implementation:
Unknown
Milestone target:
None

Related branches

Sprints

Whiteboard

Gerrit topic: https://review.openstack.org/#q,topic:bp/replace-iteritems-with-items,n,z

Addressed by: https://review.openstack.org/445856
    [1/3]Replace six.iteritems() with .items()

Addressed by: https://review.openstack.org/445865
    [2/3]Replace six.iteritems() with .items()

Addressed by: https://review.openstack.org/445878
    [3/3]Replace six.iteritems() with .items()

Gerrit topic: https://review.openstack.org/#q,topic:replace-iteritems-with-items,n,z

Addressed by: https://review.openstack.org/445273
    Replace six.iteritems() with .items()

Addressed by: https://review.openstack.org/481407
    Replace six.iteritems() with .items()

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.

Subscribers

No subscribers.