congress

Most basic DSE data structure (Node) has unsused pretty printer instance

Bug #1456729 reported by Cleber Rosa
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
congress
Fix Released
Undecided
Cleber Rosa
congress liberty-2

Bug Description

File congress/dse/amqprouter.py implements the Node class. This class gets a pretty printer that is not used anywhere else.

This deserves a cleanup.

Cleber Rosa (cleber-gnu)
Changed in congress:
assignee: nobody → Cleber Rosa (cleber-gnu)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to congress (master)

Reviewed: https://review.openstack.org/183855
Committed: https://git.openstack.org/cgit/openstack/congress/commit/?id=bdd1d93574cbff0f0a0b7e99fe79387cfad7261f
Submitter: Jenkins
Branch: master

commit bdd1d93574cbff0f0a0b7e99fe79387cfad7261f
Author: Cleber Rosa <email address hidden>
Date: Sat May 16 20:33:20 2015 -0300

    Unused pretty printer cleanup

    The ampqrouter in its most basic data structure, Node, adds a pretty
    printer that for as much as I investigated, has no real use
    whatsoever.

    Besides cleaning up the code this should also slightly improve the
    performance every time a Node class is instantiated.

    Closes-Bug: 1456729
    Change-Id: Ic8e6a9523f76d0cd5bfc7cfe816fb43884fbdf5d
    Signed-off-by: Cleber Rosa <email address hidden>

Changed in congress:
status: In Progress → Fix Committed
Tim Hinrichs (thinrichs)
Changed in congress:
milestone: none → liberty-2
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.