summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rwxr-xr-xvespaclient-core/src/main/java/com/yahoo/feedapi/SharedSender.java34
-rwxr-xr-xvespaclient-core/src/main/java/com/yahoo/feedapi/SingleSender.java12
2 files changed, 11 insertions, 35 deletions
diff --git a/vespaclient-core/src/main/java/com/yahoo/feedapi/SharedSender.java b/vespaclient-core/src/main/java/com/yahoo/feedapi/SharedSender.java
index 48ff0eae36b..97b9ec93f56 100755
--- a/vespaclient-core/src/main/java/com/yahoo/feedapi/SharedSender.java
+++ b/vespaclient-core/src/main/java/com/yahoo/feedapi/SharedSender.java
@@ -135,7 +135,7 @@ public class SharedSender implements ReplyHandler {
* @param owner A callback to send replies to when received from messagebus
*/
public void send(Message msg, ResultCallback owner) {
- send(msg, owner, -1, true);
+ send(msg, owner, true);
}
/**
@@ -144,40 +144,22 @@ public class SharedSender implements ReplyHandler {
*
* @param msg The message to send
* @param owner The callback to send replies to when received from messagebus
- * @param maxPendingPerOwner The maximum number of pending messages the callback
* @param blockingQueue If true, block until the message bus queue is available.
*/
- public void send(Message msg, ResultCallback owner, int maxPendingPerOwner, boolean blockingQueue) {
+ public void send(Message msg, ResultCallback owner, boolean blockingQueue) {
// Silently fail messages that are attempted sent after the callback aborted.
if (owner.isAborted()) {
return;
}
- try {
- synchronized (monitor) {
- if (maxPendingPerOwner != -1 && blockingQueue) {
- while (true) {
- Integer count = activeOwners.get(owner);
-
- if (count != null && count >= maxPendingPerOwner) {
- log.log(LogLevel.INFO, "Owner " + owner + " already has " + count + " pending. Waiting for replies");
- monitor.wait(10000);
- } else {
- break;
- }
- }
- }
-
- Integer count = activeOwners.get(owner);
+ synchronized (monitor) {
+ Integer count = activeOwners.get(owner);
- if (count == null) {
- activeOwners.put(owner, 1);
- } else {
- activeOwners.put(owner, count + 1);
- }
+ if (count == null) {
+ activeOwners.put(owner, 1);
+ } else {
+ activeOwners.put(owner, count + 1);
}
- } catch (InterruptedException e) {
- return;
}
msg.setContext(owner);
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 a9a08562c9d..242cbf9db17 100755
--- a/vespaclient-core/src/main/java/com/yahoo/feedapi/SingleSender.java
+++ b/vespaclient-core/src/main/java/com/yahoo/feedapi/SingleSender.java
@@ -84,19 +84,13 @@ public class SingleSender implements SimpleFeedAccess {
return m;
}
- public void send(Message m) {
- send(m, -1);
- }
-
/**
- * Sends the given message, allowing a maximum of maxPending messages to be
- * sent for this sender.
+ * Sends the given message.
*
* @param m The message to send
- * @param maxPending The number of pending messages to block on for this sender.
*/
- public void send(Message m, int maxPending) {
- sender.send(processMessage(m), owner, maxPending, blockingQueue);
+ public void send(Message m) {
+ sender.send(processMessage(m), owner, blockingQueue);
}
public void done() {