From c8ec01880b2cc8d9c9994fa3741906fd14495d6d Mon Sep 17 00:00:00 2001 From: "James T. Boylan" Date: Thu, 16 Apr 2020 11:23:12 -0500 Subject: [PATCH] Correcting the spelling in the presto.py from `principle_username` to `principal_username`. --- pyhive/presto.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/pyhive/presto.py b/pyhive/presto.py index 9b348735..aa5f29ac 100644 --- a/pyhive/presto.py +++ b/pyhive/presto.py @@ -79,7 +79,7 @@ class Cursor(common.DBAPICursor): visible by other cursors or connections. """ - def __init__(self, host, port='8080', username=None, principle_username=None, catalog='hive', + def __init__(self, host, port='8080', username=None, principal_username=None, catalog='hive', schema='default', poll_interval=1, source='pyhive', session_props=None, protocol='http', password=None, requests_session=None, requests_kwargs=None, KerberosRemoteServiceName=None, KerberosPrincipal=None, @@ -89,7 +89,7 @@ def __init__(self, host, port='8080', username=None, principle_username=None, ca :param host: hostname to connect to, e.g. ``presto.example.com`` :param port: int -- port, defaults to 8080 :param username: string -- defaults to system user name - :param principle_username: string -- defaults to ``username`` argument if it exists, + :param principal_username: string -- defaults to ``username`` argument if it exists, else defaults to system user name :param catalog: string -- defaults to ``hive`` :param schema: string -- defaults to ``default`` @@ -126,19 +126,19 @@ class will use the default requests behavior of making a new session per HTTP re """ Presto User Impersonation: https://docs.starburstdata.com/latest/security/impersonation.html - User impersonation allows the execution of queries in Presto based on principle_username + User impersonation allows the execution of queries in Presto based on principal_username argument, instead of executing the query as the account which authenticated against Presto. (Usually a service account) Allows for a service account to authenticate with Presto, and then leverage the - principle_username as the user Presto will execute the query as. This is required by + principal_username as the user Presto will execute the query as. This is required by applications that leverage authentication methods like SAML, where the application has a username, but not a password to still leverage user specific Presto Resource Groups and Authorization rules that would not be applied when only using a shared service account. This also allows auditing of who is executing a query in these environments, instead of having all queryes run by the shared service account. """ - self._username = principle_username or username or getpass.getuser() + self._username = principal_username or username or getpass.getuser() self._catalog = catalog self._schema = schema self._arraysize = 1