SNOW-2019041: Remove Dependency on joda-time (#2136) (#2139)

Author: sfc-gh-wfateem

parent-pom.xml

@@ -58,7 +58,6 @@
     <jacoco.skip.instrument>true</jacoco.skip.instrument>
     <javax.servlet.version>3.1.0</javax.servlet.version>
     <jna.version>5.13.0</jna.version>
-    <joda.time.version>2.8.1</joda.time.version>
     <json.smart.version>2.5.2</json.smart.version>
     <junit4.version>4.13.2</junit4.version>
     <junit.version>5.11.1</junit.version>
@@ -360,11 +359,6 @@
         <artifactId>javax.servlet-api</artifactId>
         <version>${javax.servlet.version}</version>
       </dependency>
-      <dependency>
-        <groupId>joda-time</groupId>
-        <artifactId>joda-time</artifactId>
-        <version>${joda.time.version}</version>
-      </dependency>
       <dependency>
         <groupId>net.minidev</groupId>
         <artifactId>json-smart</artifactId>
@@ -683,10 +677,6 @@
       <groupId>javax.servlet</groupId>
       <artifactId>javax.servlet-api</artifactId>
     </dependency>
-    <dependency>
-      <groupId>joda-time</groupId>
-      <artifactId>joda-time</artifactId>
-    </dependency>
     <dependency>
       <groupId>net.minidev</groupId>
       <artifactId>json-smart</artifactId>

src/main/java/net/snowflake/client/core/EventHandler.java

@@ -12,22 +12,18 @@
 import java.util.ArrayList;
 import java.util.Comparator;
 import java.util.Date;
-import java.util.HashMap;
-import java.util.Map;
 import java.util.TimeZone;
 import java.util.TreeSet;
 import java.util.concurrent.Executors;
 import java.util.concurrent.ScheduledExecutorService;
 import java.util.concurrent.ThreadFactory;
 import java.util.concurrent.TimeUnit;
-import java.util.concurrent.atomic.AtomicInteger;
 import java.util.logging.Formatter;
 import java.util.logging.Handler;
 import java.util.logging.LogRecord;
 import java.util.zip.GZIPOutputStream;
 import net.snowflake.client.log.SFLogger;
 import net.snowflake.client.log.SFLoggerFactory;
-import org.joda.time.DateTime;
 
 public class EventHandler extends Handler {
   private static final SFLogger logger = SFLoggerFactory.getLogger(EventHandler.class);
@@ -94,12 +90,6 @@ public void run() {
   // Period of time (in ms) to wait before waking the QueueFlusher
   private final int flushPeriodMs;
 
-  // Map to keep track of incident signatures for throttling incidents
-  private final Map<String, AtomicInteger> incidentCounter;
-
-  // Map
-  private final Map<String, DateTime> throttledIncidents;
-
   // Queue to buffer events while they are waiting to be flushed
   private final ArrayList<Event> eventBuffer;
 
@@ -117,9 +107,6 @@ public EventHandler(int maxEntries, int flushPeriodMs) {
     logBuffer = new ArrayList<>();
 
     logDumpPathPrefix = EventUtil.getDumpPathPrefix();
-
-    incidentCounter = new HashMap<>();
-    throttledIncidents = new HashMap<>();
   }
 
   /**

thin_public_pom.xml

@@ -54,7 +54,6 @@
     <jackson.version>2.18.1</jackson.version>
     <javax.servlet.version>3.1.0</javax.servlet.version>
     <jna.version>5.13.0</jna.version>
-    <joda.time.version>2.8.1</joda.time.version>
     <json.smart.version>2.5.2</json.smart.version>
     <jsoup.version>1.15.3</jsoup.version>
     <metrics.version>2.2.0</metrics.version>
@@ -234,11 +233,6 @@
       <artifactId>javax.servlet-api</artifactId>
       <version>${javax.servlet.version}</version>
     </dependency>
-    <dependency>
-      <groupId>joda-time</groupId>
-      <artifactId>joda-time</artifactId>
-      <version>${joda.time.version}</version>
-    </dependency>
     <dependency>
       <groupId>net.java.dev.jna</groupId>
       <artifactId>jna</artifactId>