Skip to content

Commit 3b6fc08

Browse files
author
Alexandru Scvortov
committed
remove confirm.select{multiple} from java API
1 parent db65727 commit 3b6fc08

File tree

5 files changed

+10
-20
lines changed

5 files changed

+10
-20
lines changed

src/com/rabbitmq/client/Channel.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -660,12 +660,10 @@ Queue.DeclareOk queueDeclare(String queue, boolean durable, boolean exclusive, b
660660

661661
/**
662662
* Enables publisher acknowledgements on this channel.
663-
* @param multiple determines whether the broker can acknowledge
664-
* multiple messages at the same time
665663
* @see com.rabbitmq.client.AMQP.Confirm.Select
666664
* @throws java.io.IOException if an error is encountered
667665
*/
668-
Confirm.SelectOk confirmSelect(boolean multiple) throws IOException;
666+
Confirm.SelectOk confirmSelect() throws IOException;
669667

670668
/**
671669
* Returns the number of messages published since the channel was

src/com/rabbitmq/client/impl/ChannelN.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -851,13 +851,13 @@ public Tx.RollbackOk txRollback()
851851
}
852852

853853
/** Public API - {@inheritDoc} */
854-
public Confirm.SelectOk confirmSelect(boolean multiple)
854+
public Confirm.SelectOk confirmSelect()
855855
throws IOException
856856
{
857857
if (publishedMessageCount.get() == -1)
858858
publishedMessageCount.set(0);
859859
return (Confirm.SelectOk)
860-
exnWrappingRpc(new Confirm.Select(multiple, false)).getMethod();
860+
exnWrappingRpc(new Confirm.Select(false)).getMethod();
861861

862862
}
863863

test/src/com/rabbitmq/client/test/functional/Confirm.java

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ public void handleAck(long seqNo,
6565
}
6666
}
6767
});
68-
channel.confirmSelect(true);
68+
channel.confirmSelect();
6969
channel.queueDeclare("confirm-test", true, true, false, null);
7070
channel.basicConsume("confirm-test", true,
7171
new DefaultConsumer(channel));
@@ -217,7 +217,7 @@ public void testConfirmSelect()
217217
{
218218
try {
219219
Channel ch = connection.createChannel();
220-
ch.confirmSelect(false);
220+
ch.confirmSelect();
221221
ch.txSelect();
222222
fail();
223223
} catch (IOException ioe) {
@@ -226,22 +226,14 @@ public void testConfirmSelect()
226226
try {
227227
Channel ch = connection.createChannel();
228228
ch.txSelect();
229-
ch.confirmSelect(false);
230-
fail();
231-
} catch (IOException ioe) {
232-
checkShutdownSignal(AMQP.PRECONDITION_FAILED, ioe);
233-
}
234-
try {
235-
Channel ch = connection.createChannel();
236-
ch.confirmSelect(true);
237-
ch.confirmSelect(false);
229+
ch.confirmSelect();
238230
fail();
239231
} catch (IOException ioe) {
240232
checkShutdownSignal(AMQP.PRECONDITION_FAILED, ioe);
241233
}
242234
Channel ch = connection.createChannel();
243-
ch.confirmSelect(true);
244-
ch.confirmSelect(true);
235+
ch.confirmSelect();
236+
ch.confirmSelect();
245237
}
246238

247239
/* Publish NUM_MESSAGES persistent messages and wait for

test/src/com/rabbitmq/examples/ConfirmDontLoseMessages.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ public void run() {
7373
Connection conn = connectionFactory.newConnection();
7474
Channel ch = conn.createChannel();
7575
ch.queueDeclare(QUEUE_NAME, true, false, true, null);
76-
ch.confirmSelect(true);
76+
ch.confirmSelect();
7777
ch.setAckListener(new AckListener() {
7878
public void handleAck(long seqNo,
7979
boolean multiple) {

test/src/com/rabbitmq/examples/MulticastMain.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ public static void main(String[] args) {
141141
producerConnections[i] = conn;
142142
Channel channel = conn.createChannel();
143143
if (producerTxSize > 0) channel.txSelect();
144-
if (pubAck) channel.confirmSelect(false);
144+
if (pubAck) channel.confirmSelect();
145145
channel.exchangeDeclare(exchangeName, exchangeType);
146146
final Producer p = new Producer(channel, exchangeName, id,
147147
flags, producerTxSize,

0 commit comments

Comments
 (0)