Landscape Client

Code review comment for lp:~al-maisan/landscape-client/obsolete-results-v

  1. obsolete-results-v
  2. Merge into trunk
Revision history for this message
Thomas Herve (therve) wrote :

[5] Some pyflakes:
landscape/broker/tests/test_exchange.py:1: 'os' imported but unused
landscape/broker/tests/test_exchange.py:8: 'ExchangeStore' imported but unused

[6] get_message_context should retrieve all the data from the database in one query, and pass it to the MessageContext __init__, thus removing the select in there.

+1!

review: Approve

« Back to merge proposal