tree 3e0d247500ec74d3fb97cf10611e64969fb5862e
parent f94d19776bfdf9e618bdee96bee1a04bf182cf0a
author Olav Loite <koloite@gmail.com> 1572351628 +0100
committer Knut Olav Løite <koloite@gmail.com> 1572420929 +0000

spanner: Re-enable and unflake TestSessionDestroy

TestSessionDestroy was disabled as it was flaky. This CL fixes
the flakiness as:
1. The session pool now uses BatchCreateSessions to immediately
   start creating the MinOpened sessions when the pool is created,
   instead of letting the maintainer do this. This guarantees that
   the initial session creation has already started when the pool
   has been created.
2. By setting MinBurst=1, the take() method is guaranteed to wait
   for the initial session creation to finish instead of creating
   a new session, as the session creation is guaranteed to already
   have been started (see 1) and MaxBurst=1 limits the number of
   sessions in parallel creation to 1.

Change-Id: I661a4a4f20bc40abc0c19a5637295afef6a51637
Reviewed-on: https://code-review.googlesource.com/c/gocloud/+/47610
Reviewed-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jean de Klerk <deklerk@google.com>