Fixes (#25)

Update libthrift version to be consist; Change TSerializer to be an object variable for thread-safety;
pinterest · Dec 16, 2019 · f795e36 · f795e36
1 parent 7e40a48
commit f795e36
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/singer-commons/pom.xml b/singer-commons/pom.xml
@@ -90,7 +90,7 @@
     <dependency>
       <groupId>org.apache.thrift</groupId>
       <artifactId>libthrift</artifactId>
-      <version>0.10.0</version>
+      <version>0.12.0</version>
     </dependency>
     <dependency>
       <groupId>org.slf4j</groupId>

diff --git a/...c/main/java/com/pinterest/singer/writer/headersinjectors/LoggingAuditHeadersInjector.java b/...c/main/java/com/pinterest/singer/writer/headersinjectors/LoggingAuditHeadersInjector.java
@@ -18,9 +18,9 @@
 public class LoggingAuditHeadersInjector implements HeadersInjector {
 
   private static final Logger LOG = LoggerFactory.getLogger(LoggingAuditHeadersInjector.class);
-
   private static final String HEADER_KEY = "loggingAuditHeaders";
-  private static final TSerializer SER = new TSerializer();
+
+  private final TSerializer SER = new TSerializer();
 
   public static String getHeaderKey() {
     return HEADER_KEY;