Revert "Also obey the timeout."

4 files changed, 8 insertions, 16 deletions

diff --git a/vespaclient-core/src/main/java/com/yahoo/feedapi/Feeder.java b/vespaclient-core/src/main/java/com/yahoo/feedapi/Feeder.java
index a1440a1ff32..e354cba141d 100644
--- a/vespaclient-core/src/main/java/com/yahoo/feedapi/Feeder.java
+++ b/vespaclient-core/src/main/java/com/yahoo/feedapi/Feeder.java
@@ -78,7 +78,7 @@ public abstract class Feeder {
             return errors;
         }
 
-        while (!sender.isAborted() && !sender.hasTimedOut()) {
+        while (!sender.isAborted()) {
             try {
                 VespaXMLFeedReader.Operation op = new VespaXMLFeedReader.Operation();
                 reader.read(op);
@@ -92,7 +92,10 @@ public abstract class Feeder {
                 } else {
                     sender.sendOperation(op);
                 }
-            } catch (XMLStreamException | NullPointerException e) {
+            } catch (XMLStreamException e) {
+                addException(e);
+                break;
+            } catch (NullPointerException e) {
                 addException(e);
                 break;
             } catch (Exception e) {
diff --git a/vespaclient-core/src/main/java/com/yahoo/feedapi/SimpleFeedAccess.java b/vespaclient-core/src/main/java/com/yahoo/feedapi/SimpleFeedAccess.java
index c4fb9a1e04a..98609650432 100755
--- a/vespaclient-core/src/main/java/com/yahoo/feedapi/SimpleFeedAccess.java
+++ b/vespaclient-core/src/main/java/com/yahoo/feedapi/SimpleFeedAccess.java
@@ -14,7 +14,6 @@ public interface SimpleFeedAccess {
     void put(Document doc, TestAndSetCondition condition);
     void remove(DocumentId docId, TestAndSetCondition condition);
     void update(DocumentUpdate update, TestAndSetCondition condition);
-    boolean hasTimedOut();
     boolean isAborted();
     void close();
 }
diff --git a/vespaclient-core/src/main/java/com/yahoo/feedapi/SingleSender.java b/vespaclient-core/src/main/java/com/yahoo/feedapi/SingleSender.java
index 6f9d539855e..e0e12b26ae6 100755
--- a/vespaclient-core/src/main/java/com/yahoo/feedapi/SingleSender.java
+++ b/vespaclient-core/src/main/java/com/yahoo/feedapi/SingleSender.java
@@ -19,18 +19,16 @@ public class SingleSender implements SimpleFeedAccess {
     private final SharedSender.ResultCallback owner;
     private final SharedSender sender;
     private final List<MessageProcessor> messageProcessors = new ArrayList<>();
-    private final long timeofDoom;
     private boolean blockingQueue;
 
-    public SingleSender(SharedSender.ResultCallback owner, SharedSender sender, long timeoutMS, boolean blockingQueue) {
+    public SingleSender(SharedSender.ResultCallback owner, SharedSender sender, boolean blockingQueue) {
         this.owner = owner;
         this.sender = sender;
-        timeofDoom = System.currentTimeMillis() + timeoutMS;
         this.blockingQueue = blockingQueue;
     }
 
-    public SingleSender(SharedSender.ResultCallback owner, int timeoutMS, SharedSender sender) {
-        this(owner, sender, timeoutMS, true);
+    public SingleSender(SharedSender.ResultCallback owner, SharedSender sender) {
+        this(owner, sender, true);
     }
 
     @Override
@@ -105,9 +103,4 @@ public class SingleSender implements SimpleFeedAccess {
 
     @Override
     public void close() { }
-
-    @Override
-    boolean hasTimedOut() {
-        return System.currentTimeMillis() > timeofDoom;
-    }
 }
diff --git a/vespaclient-core/src/main/java/com/yahoo/feedapi/VespaFeedSender.java b/vespaclient-core/src/main/java/com/yahoo/feedapi/VespaFeedSender.java
index a5042e91ef0..b441e81a829 100755
--- a/vespaclient-core/src/main/java/com/yahoo/feedapi/VespaFeedSender.java
+++ b/vespaclient-core/src/main/java/com/yahoo/feedapi/VespaFeedSender.java
@@ -17,9 +17,6 @@ public class VespaFeedSender {
     public boolean isAborted() {
         return sender.isAborted();
     }
-    boolean hasTimedOut() {
-        return sender.hasTimedOut();
-    }
 
     public void sendOperation(VespaXMLFeedReader.Operation op) {
         switch (op.getType()) {