)]}'
{"java/src/main/java/monasca/api/infrastructure/persistence/influxdb/InfluxV9MeasurementRepo.java":[{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"1ab54676784ca42900aa5990395339a28842b4c9","unresolved":false,"context_lines":[{"line_number":145,"context_line":"                             influxV9Utils.filterPrivateTags(serie.getTags()));"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"        for (String[] values : serie.getValues()) {"},{"line_number":148,"context_line":"          length \u003d values[0].length();"},{"line_number":149,"context_line":"          millisecond \u003d values[0].substring(20, length-1);"},{"line_number":150,"context_line":"          millisecond_3d \u003d StringUtils.rightPad(millisecond, 3, \u00270\u0027);"},{"line_number":151,"context_line":"          timestamp \u003d values[0].substring(0, 19) + \u0027.\u0027 + millisecond_3d + \u0027Z\u0027;"}],"source_content_type":"text/x-java","patch_set":1,"id":"5a710552_8d3b2beb","line":148,"updated":"2015-12-16 00:24:47.000000000","message":"In Java, you should always declare the variable right where you use it. So, just declare and set at the same time:\n\nint length \u003d values[0].length();\n\nand if you are me :), you do it this way:\n\nfinal int length \u003d values[0].length();","commit_id":"bdcace7ccdb810ce5b88f59e1a9b79114c1a3514"},{"author":{"_account_id":18179,"name":"Kaiyan Sheng","email":"kaiyan.sheng@hpe.com","username":"kaiyan-sheng"},"change_message_id":"fa3fe95b49abee5d7ff0d9d4f991e7b9e4821f71","unresolved":false,"context_lines":[{"line_number":145,"context_line":"                             influxV9Utils.filterPrivateTags(serie.getTags()));"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"        for (String[] values : serie.getValues()) {"},{"line_number":148,"context_line":"          length \u003d values[0].length();"},{"line_number":149,"context_line":"          millisecond \u003d values[0].substring(20, length-1);"},{"line_number":150,"context_line":"          millisecond_3d \u003d StringUtils.rightPad(millisecond, 3, \u00270\u0027);"},{"line_number":151,"context_line":"          timestamp \u003d values[0].substring(0, 19) + \u0027.\u0027 + millisecond_3d + \u0027Z\u0027;"}],"source_content_type":"text/x-java","patch_set":1,"id":"5a710552_28b54530","line":148,"in_reply_to":"5a710552_8d3b2beb","updated":"2015-12-16 00:37:32.000000000","message":"Thank you Craig! Good to know!! I will change it and post a new patch.","commit_id":"bdcace7ccdb810ce5b88f59e1a9b79114c1a3514"},{"author":{"_account_id":15027,"name":"Michael Hoppal","email":"michael.jam.hoppal@hpe.com","username":"michael-hoppal"},"change_message_id":"41e56405ed3e5c17b2f324da45b4452b91b46831","unresolved":false,"context_lines":[{"line_number":21,"context_line":"import org.joda.time.DateTime;"},{"line_number":22,"context_line":"import org.slf4j.Logger;"},{"line_number":23,"context_line":"import org.slf4j.LoggerFactory;"},{"line_number":24,"context_line":"import org.apache.commons.lang.StringUtils;"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"import java.io.IOException;"},{"line_number":27,"context_line":"import java.util.HashMap;"}],"source_content_type":"text/x-java","patch_set":2,"id":"5a710552_26ba5124","line":24,"updated":"2015-12-16 04:41:28.000000000","message":"could you put this in alphabetical as the other imports are.","commit_id":"2227c27c9bf03a85c701f2523c870c0121e2a507"}]}