Don't populate new or deprecated kwargs unless explicitly set.

Author: tswast

doc/source/whatsnew/v0.25.0.rst

@@ -191,8 +191,6 @@ Optional libraries below the lowest tested version may still work, but are not c
 +-----------------+-----------------+
 | openpyxl        | 2.4.0           |
 +-----------------+-----------------+
-| pandas-gbq      | 0.10.0          |
-+-----------------+-----------------+
 | pyarrow         | 0.9.0           |
 +-----------------+-----------------+
 | pytables        | 3.4.2           |

pandas/io/gbq.py

@@ -145,13 +145,27 @@ def read_gbq(query, project_id=None, index_col=None, col_order=None,
     """
     pandas_gbq = _try_import()
 
+    kwargs = {}
+
+    # START: new kwargs.  Don't populate unless explicitly set.
+    if use_bqstorage_api is not None:
+        kwargs["use_bqstorage_api"] = use_bqstorage_api
+    # END: new kwargs
+
+    # START: deprecated kwargs.  Don't populate unless explicitly set.
+    if verbose is not None:
+        kwargs["verbose"] = verbose
+
+    if private_key is not None:
+        kwargs["private_key"] = private_key
+    # END: deprecated kwargs
+
     return pandas_gbq.read_gbq(
         query, project_id=project_id, index_col=index_col,
         col_order=col_order, reauth=reauth,
         auth_local_webserver=auth_local_webserver, dialect=dialect,
         location=location, configuration=configuration,
-        credentials=credentials, use_bqstorage_api=use_bqstorage_api,
-        verbose=verbose, private_key=private_key)
+        credentials=credentials, **kwargs)
 
 
 def to_gbq(dataframe, destination_table, project_id=None, chunksize=None,

pandas/tests/io/test_gbq.py

@@ -1,6 +1,7 @@
 from datetime import datetime
 import os
 import platform
+from unittest import mock
 
 import numpy as np
 import pytest
@@ -10,6 +11,7 @@
 from pandas import DataFrame
 import pandas.util.testing as tm
 
+
 api_exceptions = pytest.importorskip("google.api_core.exceptions")
 bigquery = pytest.importorskip("google.cloud.bigquery")
 service_account = pytest.importorskip("google.oauth2.service_account")
@@ -90,6 +92,41 @@ def make_mixed_dataframe_v2(test_size):
                      index=range(test_size))
 
 
+@mock.patch('pandas_gbq.read_gbq')
+def test_read_gbq_with_deprecated_kwargs(mock_read_gbq):
+    private_key = object()
+    mock_read_gbq.return_value = DataFrame([[1.0]])
+    pd.read_gbq("SELECT 1", verbose=True, private_key=private_key)
+    _, kwargs = mock_read_gbq.call_args
+    assert kwargs["verbose"]
+    assert kwargs["private_key"] is private_key
+
+
+@mock.patch('pandas_gbq.read_gbq')
+def test_read_gbq_without_deprecated_kwargs(mock_read_gbq):
+    mock_read_gbq.return_value = DataFrame([[1.0]])
+    pd.read_gbq("SELECT 1")
+    _, kwargs = mock_read_gbq.call_args
+    assert "verbose" not in kwargs
+    assert "private_key" not in kwargs
+
+
+@mock.patch('pandas_gbq.read_gbq')
+def test_read_gbq_with_new_kwargs(mock_read_gbq):
+    mock_read_gbq.return_value = DataFrame([[1.0]])
+    pd.read_gbq("SELECT 1", use_bqstorage_api=True)
+    _, kwargs = mock_read_gbq.call_args
+    assert kwargs["use_bqstorage_api"]
+
+
+@mock.patch('pandas_gbq.read_gbq')
+def test_read_gbq_without_new_kwargs(mock_read_gbq):
+    mock_read_gbq.return_value = DataFrame([[1.0]])
+    pd.read_gbq("SELECT 1")
+    _, kwargs = mock_read_gbq.call_args
+    assert "use_bqstorage_api" not in kwargs
+
+
 @pytest.mark.single
 class TestToGBQIntegrationWithServiceAccountKeyPath(object):