Address review comments.

deephaven · Nov 29, 2023 · 993f9f6 · 993f9f6
1 parent f77e453
commit 993f9f6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/engine/time/src/main/java/io/deephaven/time/calendar/BusinessCalendar.java b/engine/time/src/main/java/io/deephaven/time/calendar/BusinessCalendar.java
@@ -150,8 +150,8 @@ public BusinessCalendar(final String name, final String description, final ZoneI
         this.firstValidDate = firstValidDate;
         this.lastValidDate = lastValidDate;
         this.standardBusinessDay = standardBusinessDay;
-        this.weekendDays = weekendDays;
-        this.holidays = holidays;
+        this.weekendDays = Set.copyOf(weekendDays);
+        this.holidays = Map.copyOf(holidays);
         populateSchedules();
         populateCachedYearData();
     }

diff --git a/engine/time/src/main/java/io/deephaven/time/calendar/Calendars.java b/engine/time/src/main/java/io/deephaven/time/calendar/Calendars.java
@@ -67,7 +67,8 @@ private static void load(final String businessCalendarConfig) throws NoSuchFileE
                 final InputStream inputStream = Calendars.class.getResourceAsStream(filePath);
                 if (inputStream != null) {
                     final File calendarFile = inputStreamToFile(inputStream);
-                    final BusinessCalendar businessCalendar = BusinessCalendarXMLParser.loadBusinessCalendar(calendarFile);
+                    final BusinessCalendar businessCalendar =
+                            BusinessCalendarXMLParser.loadBusinessCalendar(calendarFile);
                     addCalendar(businessCalendar);
                     // noinspection ResultOfMethodCallIgnored
                     calendarFile.delete();