BrokerMetadata throwing TypeError using old-style string conversion

[jeffwidman]

I was accidentally passing a BrokerMetada object rather than a node_id, but debugging was more cryptic than it needed to be:

    354
    355             if conn is None:
--> 356                 assert broker, 'Broker id %s not in current metadata' % node_id
    357
    358                 log.debug("Initiating connection to node %s at %s:%s",

TypeError: not all arguments converted during string formatting

Looks like the old-style %s is having problems but using format() doesn't...

BrokerMetadata(nodeId=3, host=u'kafka03.stg.local', port=9092, rack=None)
ipdb> str(node_id)
"BrokerMetadata(nodeId=3, host=u'kafka03.stg.local', port=9092, rack=None)"
ipdb> assert False, "%s" % node_id
*** TypeError: not all arguments converted during string formatting
ipdb> assert False, "{}".format(node_id)
*** AssertionError: BrokerMetadata(nodeId=3, host=u'kafka03.stg.local', port=9092, rack=None)

[jeffwidman]

BrokerMetadata is a namedtuple: https://stackoverflow.com/questions/1455602/printing-tuple-with-string-formatting-in-python