perf(dbapi): set headers correctly for dynamic routing (#644)

Author: larkee

google/cloud/spanner_dbapi/connection.py

@@ -531,7 +531,9 @@ def connect(
  """
 
  client_info = ClientInfo(
- user_agent=user_agent or DEFAULT_USER_AGENT, python_version=PY_VERSION
+ user_agent=user_agent or DEFAULT_USER_AGENT,
+ python_version=PY_VERSION,
+ client_library_version=spanner.__version__,
  )
 
  if isinstance(credentials, str):

google/cloud/spanner_dbapi/version.py

@@ -17,4 +17,4 @@
 
 PY_VERSION = platform.python_version()
 VERSION = pkg_resources.get_distribution("google-cloud-spanner").version
-DEFAULT_USER_AGENT = "dbapi/" + VERSION
+DEFAULT_USER_AGENT = "gl-dbapi/" + VERSION

tests/system/test_dbapi.py

@@ -409,7 +409,11 @@ def test_user_agent(shared_instance, dbapi_database):
  conn = connect(shared_instance.name, dbapi_database.name)
  assert (
  conn.instance._client._client_info.user_agent
- == "dbapi/" + pkg_resources.get_distribution("google-cloud-spanner").version
+ == "gl-dbapi/" + pkg_resources.get_distribution("google-cloud-spanner").version
+ )
+ assert (
+ conn.instance._client._client_info.client_library_version
+ == pkg_resources.get_distribution("google-cloud-spanner").version
  )