Skip to content

Commit

Permalink
Update default_module_mapping.py to add snowflake's snowpark (pantsbu…
Browse files Browse the repository at this point in the history
…ild#19671)

Very common in the snowflake world. Deserves a default mapping, imo.

Tested locally with an entry in BUILD like:
```python
python_requirements(
    name="dag",
    resolve="dag",
    source="dag-requirements.txt",
    module_mapping={
        "snowflake-snowpark-python": ("snowflake.snowpark",),
        "google-cloud-datastream": ("google.cloud.datastream",),
        "google-cloud-spanner": ("google.cloud.spanner",),
    }
)
```
  • Loading branch information
ryaminal authored Aug 26, 2023
1 parent ee6f123 commit 9283f74
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -98,13 +98,15 @@
"google-cloud-bigquery": ("google.cloud.bigquery",),
"google-cloud-bigtable": ("google.cloud.bigtable",),
"google-cloud-datastore": ("google.cloud.datastore",),
"google-cloud-datastream": ("google.cloud.datastream",),
"google-cloud-firestore": ("google.cloud.firestore",),
"google-cloud-functions": ("google.cloud.functions_v1", "google.cloud.functions"),
"google-cloud-iam": ("google.cloud.iam_credentials_v1",),
"google-cloud-iot": ("google.cloud.iot_v1",),
"google-cloud-logging": ("google.cloud.logging_v2", "google.cloud.logging"),
"google-cloud-pubsub": ("google.cloud.pubsub_v1", "google.cloud.pubsub"),
"google-cloud-secret-manager": ("google.cloud.secretmanager",),
"google-cloud-spanner": ("google.cloud.spanner",),
"google-cloud-storage": ("google.cloud.storage",),
"graphql-core": ("graphql",),
"grpcio": ("grpc",),
Expand Down Expand Up @@ -238,6 +240,7 @@
"sseclient-py": ("sseclient",),
"setuptools": ("easy_install", "pkg_resources", "setuptools"),
"snowflake-connector-python": ("snowflake.connector",),
"snowflake-snowpark-python": ("snowflake.snowpark",),
"snowflake-sqlalchemy": ("snowflake.sqlalchemy",),
"strawberry-graphql": ("strawberry",),
"streamlit-aggrid": ("st_aggrid",),
Expand Down

0 comments on commit 9283f74

Please sign in to comment.