SPY-164: Clone gets correctly.

Motivation
----------
When a gets operation is cloned, it currently gets incorrectly cloned into
a regular get operation. This can cause issues later on.

Modifications
-------------
Make sure that a gets() gets correctly cloned into a new gets().

Result
------
The cloning is now properly done, based on the operation type.

Thanks to Brad Svee (@sveesible) for the fix suggestion.

Change-Id: I7b8820bea20d0a0d9555f1bb9b2a46e5327a5f78
Reviewed-on: http://review.couchbase.org/36221
Reviewed-by: Matt Ingenthron <matt@couchbase.com>
Tested-by: Michael Nitschinger <michael.nitschinger@couchbase.com>

Author: daschl

src/main/java/net/spy/memcached/protocol/binary/BinaryOperationFactory.java

@@ -23,17 +23,46 @@
 
 package net.spy.memcached.protocol.binary;
 
-import java.util.ArrayList;
-import java.util.Collection;
-import java.util.Map;
-
-import javax.security.auth.callback.CallbackHandler;
-
-import net.spy.memcached.ops.*;
+import net.spy.memcached.ops.BaseOperationFactory;
+import net.spy.memcached.ops.CASOperation;
+import net.spy.memcached.ops.ConcatenationOperation;
+import net.spy.memcached.ops.ConcatenationType;
+import net.spy.memcached.ops.DeleteOperation;
+import net.spy.memcached.ops.FlushOperation;
+import net.spy.memcached.ops.GetAndTouchOperation;
+import net.spy.memcached.ops.GetOperation;
 import net.spy.memcached.ops.GetOperation.Callback;
+import net.spy.memcached.ops.GetlOperation;
+import net.spy.memcached.ops.GetsOperation;
+import net.spy.memcached.ops.KeyedOperation;
+import net.spy.memcached.ops.MultiGetOperationCallback;
+import net.spy.memcached.ops.MultiGetsOperationCallback;
+import net.spy.memcached.ops.MultiReplicaGetOperationCallback;
+import net.spy.memcached.ops.Mutator;
+import net.spy.memcached.ops.MutatorOperation;
+import net.spy.memcached.ops.NoopOperation;
+import net.spy.memcached.ops.ObserveOperation;
+import net.spy.memcached.ops.Operation;
+import net.spy.memcached.ops.OperationCallback;
+import net.spy.memcached.ops.ReplicaGetOperation;
+import net.spy.memcached.ops.ReplicaGetsOperation;
+import net.spy.memcached.ops.SASLAuthOperation;
+import net.spy.memcached.ops.SASLMechsOperation;
+import net.spy.memcached.ops.SASLStepOperation;
+import net.spy.memcached.ops.StatsOperation;
+import net.spy.memcached.ops.StoreOperation;
+import net.spy.memcached.ops.StoreType;
+import net.spy.memcached.ops.TapOperation;
+import net.spy.memcached.ops.UnlockOperation;
+import net.spy.memcached.ops.VersionOperation;
 import net.spy.memcached.tapmessage.RequestMessage;
 import net.spy.memcached.tapmessage.TapOpcode;
 
+import javax.security.auth.callback.CallbackHandler;
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.Map;
+
 /**
  * Factory for binary operations.
  */
@@ -154,7 +183,7 @@ protected Collection<? extends Operation> cloneGet(KeyedOperation op) {
       if(getCb != null) {
         rv.add(get(k, getCb));
       } else if(getsCb != null) {
-        rv.add(get(k, getCb));
+        rv.add(gets(k, getsCb));
       } else {
         rv.add(replicaGet(k, ((ReplicaGetOperationImpl)op).getReplicaIndex() ,replicaGetCb));
       }