Skip to content

Commit e71b5a3

Browse files
committed
[SPARK-27624][CORE] Fix CalenderInterval to show an empty interval correctly
1 parent 7a8cc8e commit e71b5a3

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

common/unsafe/src/main/java/org/apache/spark/unsafe/types/CalendarInterval.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -321,6 +321,10 @@ public String toString() {
321321
appendUnit(sb, rest, "microsecond");
322322
}
323323

324+
if (months == 0 && microseconds == 0) {
325+
sb.append(" 0 microseconds");
326+
}
327+
324328
return sb.toString();
325329
}
326330

common/unsafe/src/test/java/org/apache/spark/unsafe/types/CalendarIntervalSuite.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,9 @@ public void equalsTest() {
4141
public void toStringTest() {
4242
CalendarInterval i;
4343

44+
i = new CalendarInterval(0, 0);
45+
assertEquals("interval 0 microseconds", i.toString());
46+
4447
i = new CalendarInterval(34, 0);
4548
assertEquals("interval 2 years 10 months", i.toString());
4649

0 commit comments

Comments
 (0)