SCBC-492: getAllReplicas and lookupInAllReplicas now require timeouts

Fixing a 3.9.0 regression.

Change-Id: Ia0af4b53b061b5d32330183fbc012aff8545a327
Reviewed-on: https://review.couchbase.org/c/couchbase-jvm-clients/+/232106
Tested-by: Build Bot <[email protected]>
Reviewed-by: David Nault <[email protected]>
(cherry picked from commit 11483fe)
Reviewed-on: https://review.couchbase.org/c/couchbase-jvm-clients/+/233200
Reviewed-by: Graham Pople <[email protected]>
Tested-by: Graham Pople <[email protected]>

Author: programmatix

scala-client/src/main/scala-2/com/couchbase/client/scala/AsyncCollection.scala

@@ -500,7 +500,8 @@ class AsyncCollection(
 
     import scala.jdk.CollectionConverters._
 
-    val results = scala.concurrent.Await.result(futureList, options.timeout)
+    val awaitTimeout = if (options.timeout == Duration.MinusInf) kvReadTimeout else options.timeout
+    val results      = scala.concurrent.Await.result(futureList, awaitTimeout)
     results.asScala.toSeq.map(res =>
       Future.successful(convertReplica(res, environment, options.transcoder))
     )
@@ -600,7 +601,8 @@ class AsyncCollection(
 
     import scala.jdk.CollectionConverters._
 
-    val results = scala.concurrent.Await.result(futureList, options.timeout)
+    val awaitTimeout = if (options.timeout == Duration.MinusInf) kvReadTimeout else options.timeout
+    val results      = scala.concurrent.Await.result(futureList, awaitTimeout)
     results.asScala.toSeq.map(res => Future.successful(convertLookupInReplica(res, environment)))
   }