All APIs should be lazy

[rfkm]

When wrapping a CompletableFuture with a Task, Task.from is often used in this library. However, this means that the CompletableFuture will be executed even if the Task is not evaluated (i.e., it is not referential transparent). This is not the expected behavior for many users.

e.g.

monix-connect/s3/src/main/scala/monix/connect/s3/S3.scala

Lines 714 to 715 in cb3f2c2

	def deleteObject(request: DeleteObjectRequest): Task[DeleteObjectResponse] =
	Task.from(s3Client.deleteObject(request))

I think it is correct to use Task.fromFutureLike.

[paualarco]

Actually, from is implicitly using fromFutureLike via fromAnyFutureViaLift.
But I think that your point is still valid and to make the referential transparent we should wrap it with Task.defer

[rfkm]

Yeah, my point was that fromFutureLike is a less error-prone API because it takes Task[F[A]].