ibis icon indicating copy to clipboard operation
ibis copied to clipboard

Fix ibis connect duckdb when pass config

Open SothanaV opened this issue 9 months ago • 2 comments

Description of changes

  • change pass config to duckdb if use ibis duckdb as a backend

SothanaV avatar May 16 '24 10:05 SothanaV

ACTION NEEDED

Ibis follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message.

Please update your PR title and description to match the specification.

github-actions[bot] avatar May 16 '24 10:05 github-actions[bot]

This PR does not have a description that matches what it does nor describes why it's necessary. The TOML change also doesn't have any stated motivation.

cpcloud avatar May 16 '24 11:05 cpcloud

Closing this for now. We can't accept this PR as is:

  1. The description doesn't match the changes.
  2. Loading the httpfs extension is not something we should do at at startup.
  3. Executing each individual configuration SET as a separate query is also nothing something we should do at startup.

cpcloud avatar May 18 '24 13:05 cpcloud