dlt-hub / dlt

data load tool (dlt) is an open source Python library that makes data loading easy 🛠️
https://dlthub.com/docs
Apache License 2.0
2.65k stars 176 forks source link

Update documentation for backend_kwargs in sql_database function for pyarrow backend #1901

Closed dat-a-man closed 1 month ago

dat-a-man commented 1 month ago

Docs update: sql_database backend_kwargs function configuration for pyarrow backend

Description

The current documentation for the sql_database function does not adequately cover the required configuration fields, specifically the tz field, which is necessary when using the pyarrow backend.

Slack:

Steps to Reproduce

Expected Behavior

The documentation should clearly state that the tz field is required when using the pyarrow backend and provide examples of how to include it in the backend_kwargs.

Proposed Solution

To resolve this issue, the documentation should be updated to include the following: Required Configuration Fields Mention that the tz field is required when using the pyarrow backend.

Example Usage

Provide an example of how to include the tz field in the backend_kwargs, such as:

sql_database(..., backend_kwargs={"tz": "UTC"})

Supported Values for tz

List examples of supported time zone strings, such as:

tz='America/New_York'
tz='+07:30'
tz='UTC'

Related Discussion

This issue was discussed in the following thread: Link to the discussion thread.

Todo

Update the documentation to reflect the necessary configuration for the pyarrow backend, ensuring that users are aware of the required tz field and how to properly configure it.