fix:reverting a change temporarily

vibhatha · Sep 18, 2024 · 1354ad0 · 1354ad0
1 parent 5b85701
commit 1354ad0
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/java/flight/flight-core/src/main/java/org/apache/arrow/flight/ServerSessionMiddleware.java b/java/flight/flight-core/src/main/java/org/apache/arrow/flight/ServerSessionMiddleware.java
@@ -16,10 +16,8 @@
  */
 package org.apache.arrow.flight;
 
-import com.google.common.base.Splitter;
 import java.util.Collections;
 import java.util.HashMap;
-import java.util.List;
 import java.util.Map;
 import java.util.concurrent.Callable;
 import java.util.concurrent.ConcurrentHashMap;
@@ -77,6 +75,7 @@ private void closeSession(String id) {
     }
 
     @Override
+    @SuppressWarnings("StringSplitter")
     public ServerSessionMiddleware onCallStarted(
         CallInfo callInfo, CallHeaders incomingHeaders, RequestContext context) {
       String sessionId = null;
@@ -85,15 +84,15 @@ public ServerSessionMiddleware onCallStarted(
       if (it != null) {
         findIdCookie:
         for (final String headerValue : it) {
-          for (final String cookie : Splitter.on(" ;").split(headerValue)) {
-            final List<String> cookiePair = Splitter.on('=').splitToList(cookie);
-            if (cookiePair.size() != 2) {
+          for (final String cookie : headerValue.split(" ;")) {
+            final String[] cookiePair = cookie.split("=");
+            if (cookiePair.length != 2) {
               // Soft failure:  Ignore invalid cookie list field
               break;
             }
 
-            if (sessionCookieName.equals(cookiePair.get(0)) && !cookiePair.get(1).isEmpty()) {
-              sessionId = cookiePair.get(1);
+            if (sessionCookieName.equals(cookiePair[0]) && !cookiePair[1].isEmpty()) {
+              sessionId = cookiePair[1];
               break findIdCookie;
             }
           }