Skip to content
This repository has been archived by the owner on Dec 18, 2023. It is now read-only.

Commit

Permalink
fix default value for password & method
Browse files Browse the repository at this point in the history
  • Loading branch information
Kyle Wigley committed Feb 16, 2021
1 parent 1188326 commit e0a457b
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 12 deletions.
20 changes: 10 additions & 10 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,16 +14,16 @@ $ pip install dbt-presto

A dbt profile can be configured to run against Presto using the following configuration:

| Option | Description | Required? | Example |
|---------|----------------------------------------------------|-------------------------|--------------------------|
| method | The Presto authentication method to use | Optional (default=`none`) | `none`|`kerberos` |
| user | Username for authentication | Required | `none`|`drew` |
| password | Password for authentication | Optional (required if `method` is `ldap|kerberos`) | `none`|`abc123` |
| database | Specify the database to build models into | Required | `analytics` |
| schema | Specify the schema to build models into | Required | `dbt_drew` |
| host | The hostname to connect to | Required | `127.0.0.1` |
| port | The port to connect to the host on | Required | `8080` |
| threads | How many threads dbt should use | Optional(default=`1`) | `8` |
| Option | Description | Required? | Example |
| -------- | ----------------------------------------- | -------------------------------------------------- | ------------------ |
| method | The Presto authentication method to use | Optional (default=`none`) | `none`\|`kerberos` |
| user | Username for authentication | Required | `drew` |
| password | Password for authentication | Optional (required if `method` is `ldap|kerberos`) | `none`\|`abc123` |
| database | Specify the database to build models into | Required | `analytics` |
| schema | Specify the schema to build models into | Required | `dbt_drew` |
| host | The hostname to connect to | Required | `127.0.0.1` |
| port | The port to connect to the host on | Required | `8080` |
| threads | How many threads dbt should use | Optional(default=`1`) | `8` |



Expand Down
4 changes: 2 additions & 2 deletions dbt/adapters/presto/connections.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@ class PrestoCredentials(Credentials):
host: str
port: Port
user: str
password: Optional[str]
method: Optional[str]
password: Optional[str] = None
method: Optional[str] = None
_ALIASES = {
'catalog': 'database'
}
Expand Down

0 comments on commit e0a457b

Please sign in to comment.