)]}'
{"thresh/src/main/java/monasca/thresh/infrastructure/persistence/AlarmDAOImpl.java":[{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"4bd87636c0cf128184fd3f359af074ebb25affd7","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"  private final DBI db;"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"  private final SimpleDateFormat simpleDateFormat;"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"  @Inject"},{"line_number":62,"context_line":"  public AlarmDAOImpl(DBI db) {"}],"source_content_type":"text/x-java","patch_set":4,"id":"1aa78d24_c659fffe","line":59,"range":{"start_line":59,"start_character":33,"end_line":59,"end_character":49},"updated":"2016-07-12 22:39:58.000000000","message":"SimpleDateFormat is not thread safe. Unbelievable, I know, but still true","commit_id":"c5af0b7e3cbdc0868522566208eafb72cad77983"}],"thresh/src/test/java/monasca/thresh/infrastructure/persistence/AlarmDAOImplTest.java":[{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"4bd87636c0cf128184fd3f359af074ebb25affd7","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"    dao.createAlarm(newAlarm);"},{"line_number":211,"context_line":"    dao.updateState(newAlarm.getId(), AlarmState.ALARM, System.currentTimeMillis());"},{"line_number":212,"context_line":"    assertEquals(dao.findById(newAlarm.getId()).getState(), AlarmState.ALARM);"},{"line_number":213,"context_line":"  }"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"  public void shouldUpdate() {"}],"source_content_type":"text/x-java","patch_set":4,"id":"1aa78d24_464fcf33","line":212,"range":{"start_line":212,"start_character":60,"end_line":212,"end_character":76},"updated":"2016-07-12 22:39:58.000000000","message":"You should also verify the time, if you can.","commit_id":"c5af0b7e3cbdc0868522566208eafb72cad77983"},{"author":{"_account_id":15027,"name":"Michael Hoppal","email":"michael.jam.hoppal@hpe.com","username":"michael-hoppal"},"change_message_id":"09c977576ef054dd0f09c9f994f9cc81de874007","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"    dao.createAlarm(newAlarm);"},{"line_number":211,"context_line":"    dao.updateState(newAlarm.getId(), AlarmState.ALARM, System.currentTimeMillis());"},{"line_number":212,"context_line":"    assertEquals(dao.findById(newAlarm.getId()).getState(), AlarmState.ALARM);"},{"line_number":213,"context_line":"  }"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"  public void shouldUpdate() {"}],"source_content_type":"text/x-java","patch_set":4,"id":"1aa78d24_dd836554","line":212,"range":{"start_line":212,"start_character":60,"end_line":212,"end_character":76},"in_reply_to":"1aa78d24_464fcf33","updated":"2016-07-13 14:17:01.000000000","message":"it is not in the alarm object or attainable by any of the dao methods let me see what i can do","commit_id":"c5af0b7e3cbdc0868522566208eafb72cad77983"}]}
