)]}'
{"java/src/main/java/monasca/api/infrastructure/persistence/hibernate/AlarmDefinitionSqlRepoImpl.java":[{"author":{"_account_id":16222,"name":"witek","email":"witold.bedyk@suse.com","username":"witek"},"change_message_id":"bb8b431c5564574e695ee57ec09810aa2f2f06c4","unresolved":false,"context_lines":[{"line_number":263,"context_line":""},{"line_number":264,"context_line":"    if (limit \u003e 0) {"},{"line_number":265,"context_line":"      limitOffset.append(\" limit :limit\");"},{"line_number":266,"context_line":"    }"},{"line_number":267,"context_line":"    if (offset !\u003d null) {"},{"line_number":268,"context_line":"      limitOffset.append(\" offset \");"},{"line_number":269,"context_line":"      limitOffset.append(offset);"}],"source_content_type":"text/x-java","patch_set":2,"id":"7aa08908_e3cac6a0","line":266,"updated":"2016-06-06 12:57:40.000000000","message":"Blank line please.","commit_id":"db92a986a1d91242352919f9c6d08170b7c30a0d"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"300cdad4f3c35a8ab2c83d1f0fcc39cec8fbdc66","unresolved":false,"context_lines":[{"line_number":263,"context_line":""},{"line_number":264,"context_line":"    if (limit \u003e 0) {"},{"line_number":265,"context_line":"      limitOffset.append(\" limit :limit\");"},{"line_number":266,"context_line":"    }"},{"line_number":267,"context_line":"    if (offset !\u003d null) {"},{"line_number":268,"context_line":"      limitOffset.append(\" offset \");"},{"line_number":269,"context_line":"      limitOffset.append(offset);"}],"source_content_type":"text/x-java","patch_set":2,"id":"7aa08908_91fde53c","line":266,"in_reply_to":"7aa08908_e3cac6a0","updated":"2016-06-07 03:59:21.000000000","message":"Done","commit_id":"db92a986a1d91242352919f9c6d08170b7c30a0d"},{"author":{"_account_id":16222,"name":"witek","email":"witold.bedyk@suse.com","username":"witek"},"change_message_id":"bb8b431c5564574e695ee57ec09810aa2f2f06c4","unresolved":false,"context_lines":[{"line_number":265,"context_line":"      limitOffset.append(\" limit :limit\");"},{"line_number":266,"context_line":"    }"},{"line_number":267,"context_line":"    if (offset !\u003d null) {"},{"line_number":268,"context_line":"      limitOffset.append(\" offset \");"},{"line_number":269,"context_line":"      limitOffset.append(offset);"},{"line_number":270,"context_line":"      limitOffset.append(\u0027 \u0027);"},{"line_number":271,"context_line":"    }"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"    String sql \u003d String.format(query, SubAlarmDefinitionQueries.buildJoinClauseFor(dimensions), sbWhere, limitOffset);"}],"source_content_type":"text/x-java","patch_set":2,"id":"7aa08908_a3371eaf","line":270,"range":{"start_line":268,"start_character":6,"end_line":270,"end_character":30},"updated":"2016-06-06 12:57:40.000000000","message":"Why don\u0027t you want to use the named parameter here?","commit_id":"db92a986a1d91242352919f9c6d08170b7c30a0d"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"04839983f9ed22805a5cbb9d512a81eff2b57a23","unresolved":false,"context_lines":[{"line_number":265,"context_line":"      limitOffset.append(\" limit :limit\");"},{"line_number":266,"context_line":"    }"},{"line_number":267,"context_line":"    if (offset !\u003d null) {"},{"line_number":268,"context_line":"      limitOffset.append(\" offset \");"},{"line_number":269,"context_line":"      limitOffset.append(offset);"},{"line_number":270,"context_line":"      limitOffset.append(\u0027 \u0027);"},{"line_number":271,"context_line":"    }"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"    String sql \u003d String.format(query, SubAlarmDefinitionQueries.buildJoinClauseFor(dimensions), sbWhere, limitOffset);"}],"source_content_type":"text/x-java","patch_set":2,"id":"7aa08908_f170a9de","line":270,"range":{"start_line":268,"start_character":6,"end_line":270,"end_character":30},"in_reply_to":"7aa08908_31feb138","updated":"2016-06-07 04:04:36.000000000","message":"But I think I might have done this wrong on previous occasion.","commit_id":"db92a986a1d91242352919f9c6d08170b7c30a0d"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"300cdad4f3c35a8ab2c83d1f0fcc39cec8fbdc66","unresolved":false,"context_lines":[{"line_number":265,"context_line":"      limitOffset.append(\" limit :limit\");"},{"line_number":266,"context_line":"    }"},{"line_number":267,"context_line":"    if (offset !\u003d null) {"},{"line_number":268,"context_line":"      limitOffset.append(\" offset \");"},{"line_number":269,"context_line":"      limitOffset.append(offset);"},{"line_number":270,"context_line":"      limitOffset.append(\u0027 \u0027);"},{"line_number":271,"context_line":"    }"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"    String sql \u003d String.format(query, SubAlarmDefinitionQueries.buildJoinClauseFor(dimensions), sbWhere, limitOffset);"}],"source_content_type":"text/x-java","patch_set":2,"id":"7aa08908_31feb138","line":270,"range":{"start_line":268,"start_character":6,"end_line":270,"end_character":30},"in_reply_to":"7aa08908_a3371eaf","updated":"2016-06-07 03:59:21.000000000","message":"I\u0027ve been having some troubles using that as named parameter, it may be that it is not possible to specify offset that way.\n\nYou know, basically it would be the best to have full criteria code here but unfortunately that would require a lot of custom association modelling and even now I am not that sure it would work.","commit_id":"db92a986a1d91242352919f9c6d08170b7c30a0d"}]}