)]}'
{"src/main/java/monasca/api/infrastructure/persistence/influxdb/Utils.java":[{"author":{"_account_id":12149,"name":"Dexter Fryar","email":"dexter.fryar@protonmail.com","username":"dlfryar"},"change_message_id":"33e5fdf21610ab8ce6fe82034744ab98dfca8fbc","unresolved":false,"context_lines":[{"line_number":93,"context_line":"      regex.append(urlEncodeUTF8(name));"},{"line_number":94,"context_line":"      regex.append(\"(\u0026|$)\");"},{"line_number":95,"context_line":"    }"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    // Dimensions are optional."},{"line_number":98,"context_line":"    if (dimensions !\u003d null \u0026\u0026 !dimensions.isEmpty()) {"},{"line_number":99,"context_line":""}],"source_content_type":"text/x-java","patch_set":2,"id":"3a961159_e1728c49","line":96,"updated":"2015-01-09 17:45:21.000000000","message":"Any thoughts on using the Apache Commons StringUtils library calling\n\nif (!dimensions.isBlank()) {\n\nin place of\n\nif (dimensions !\u003d null \u0026\u0026 !dimensions.isEmpty()) {\n\nThis would catch the condition where the dimension could have white space in it.  Not sure if .strip() is already being called on the dimension insert which would negate the need to do this.","commit_id":"1c3f4465c5900c7dad04060b2a8277922cf1f2fe"},{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"577be7aada0edc92a53c3139fddbe5c73a0c558f","unresolved":false,"context_lines":[{"line_number":93,"context_line":"      regex.append(urlEncodeUTF8(name));"},{"line_number":94,"context_line":"      regex.append(\"(\u0026|$)\");"},{"line_number":95,"context_line":"    }"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    // Dimensions are optional."},{"line_number":98,"context_line":"    if (dimensions !\u003d null \u0026\u0026 !dimensions.isEmpty()) {"},{"line_number":99,"context_line":""}],"source_content_type":"text/x-java","patch_set":2,"id":"3a961159_4197e0eb","line":96,"in_reply_to":"3a961159_e1728c49","updated":"2015-01-09 17:56:36.000000000","message":"Dimensions is a Map, not a String so can\u0027t use StringUtils.","commit_id":"1c3f4465c5900c7dad04060b2a8277922cf1f2fe"},{"author":{"_account_id":12512,"name":"Deklan Dieterly","email":"deklan.dieterly@hpe.com","username":"deklan"},"change_message_id":"f16b48eaa827af3ee3dc9b0538e3bf2adfebe376","unresolved":false,"context_lines":[{"line_number":93,"context_line":"      regex.append(urlEncodeUTF8(name));"},{"line_number":94,"context_line":"      regex.append(\"(\u0026|$)\");"},{"line_number":95,"context_line":"    }"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    // Dimensions are optional."},{"line_number":98,"context_line":"    if (dimensions !\u003d null \u0026\u0026 !dimensions.isEmpty()) {"},{"line_number":99,"context_line":""}],"source_content_type":"text/x-java","patch_set":2,"id":"3a961159_61e4dce0","line":96,"in_reply_to":"3a961159_e1728c49","updated":"2015-01-09 17:49:01.000000000","message":"We use the google guava library. it has a similar function for strings.","commit_id":"1c3f4465c5900c7dad04060b2a8277922cf1f2fe"}]}
