Skip to content

Commit e4242e4

Browse files
authored
Removed string concatenation in logging (#633)
* removed string concatenation in logging * improved unit tests assertions * Revert "improved unit tests assertions" This reverts commit 96b9a7b. * removed unnecessary logging
1 parent 770d5dd commit e4242e4

2 files changed

Lines changed: 1 addition & 4 deletions

File tree

src/main/java/com/teragrep/pth_10/ast/time/RelativeTimeParser.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ public RelativeTimestamp parse(String timestamp) {
112112
// split the timestamp to offset (before @) and snap-to-time (after @)
113113
String offsetTimestamp = timestamp.substring(0, timestamp.indexOf("@"));
114114
String snapTimestamp = timestamp.substring(timestamp.indexOf('@') + 1);
115-
LOGGER.info("Snaptime=" + snapTimestamp);
115+
LOGGER.info("Snaptime= <{}>", snapTimestamp);
116116

117117
offset = parseRelativeOffset(offsetTimestamp);
118118
snap = parseSnapToTime(snapTimestamp);

src/test/java/com/teragrep/pth_10/CatalystVisitorTest.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -138,9 +138,6 @@ void searchQueryWithTimestampTest() {
138138
DPLParserCatalystContext ctx = this.streamingTestUtil.getCtx();
139139

140140
String result = ctx.getSparkQuery();
141-
LOGGER.info("Query=" + q);
142-
LOGGER.info("Expected=" + e);
143-
LOGGER.info("Result=" + result);
144141
Assertions.assertEquals(e, result);
145142
}
146143
catch (ParseException e) {

0 commit comments

Comments
 (0)