Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[duckdb] SUMMARIZE tbl; doesn't return anything #836

Closed
alejandrob17 opened this issue Aug 27, 2023 · 2 comments · Fixed by #841
Closed

[duckdb] SUMMARIZE tbl; doesn't return anything #836

alejandrob17 opened this issue Aug 27, 2023 · 2 comments · Fixed by #841
Assignees

Comments

@alejandrob17
Copy link

What happens?

When the following command is executed:

%sql SUMMARIZE tbl;

using DuckDB, no results are being returned. Unlike previous versions, where the expected results were returned as outlined in the documentation https://duckdb.org/docs/guides/meta/summarize, the current version displays a progress bar and yields an empty dataset with a column named "Success".

To Reproduce

import duckdb
import pandas as pd

%load_ext sql
%config SqlMagic.autopandas = True
%config SqlMagic.feedback = False
%config SqlMagic.displaycon = False

%sql duckdb:///:default:
%%sql 
CREATE OR REPLACE TABLE t1(
    id INTEGER,
    x INTEGER);

%%sql
INSERT INTO t1 VALUES (1, 5), (2, -1), (3, 101);

%sql SUMMARIZE t1;

OS:

wsl2

JupySQL Version:

0.10.0

Full Name:

Alejandro Bolaños

Affiliation:

Universidad Buenos Aires

@edublancas
Copy link

@bbeat2782 can you check this? I think this might be related to some scenarios where we have to check the type of statement the user is running, we might not be correctly handling the SUMMARIZE command. see: https://github.com/search?q=repo%3Aploomber%2Fjupysql%20is_select&type=code

@edublancas
Copy link

hey @alejandrob17 we've merged a PR that fixes it, if you get a chance, please install JupySQL from the master branch and try it out. we'll release this next week!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants