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

utf8 char didn't show correctly when use pragma enable_profiling in windows. #2761

Closed
2 tasks done
l1t1 opened this issue Dec 10, 2021 · 1 comment · Fixed by #2773
Closed
2 tasks done

utf8 char didn't show correctly when use pragma enable_profiling in windows. #2761

l1t1 opened this issue Dec 10, 2021 · 1 comment · Fixed by #2773

Comments

@l1t1
Copy link

l1t1 commented Dec 10, 2021

What happens?

utf8 char didn't show correctly when use pragma enable_profiling in windows 7 Chinese version .
btw, explain command shows correct UTF8 char.

To Reproduce

create table ints as select cast(random() * 99999999 as int) i from range(100000000);
pragma enable_profiling;
select last_value(i) over () from (select i from ints order by i) sq limit 1; -- same with first_value
Result:

Environment (please complete the following information):

  • OS: [e.g. iOS]windows7 x64
  • DuckDB Version: [e.g. 22]0.3.2 dev-479
  • DuckDB Client: [e.g. Python]CLI

Before Submitting

  • Have you tried this on the latest master branch?
  • Python: pip install duckdb --upgrade --pre
  • R: install.packages("https://github.com/duckdb/duckdb/releases/download/master-builds/duckdb_r_src.tar.gz", repos = NULL)
  • Other Platforms: You can find binaries here or compile from source.
  • Have you tried the steps to reproduce? Do they include all relevant data and configuration? Does the issue you report still appear there?
@Mytherin
Copy link
Collaborator

Thanks for the report! I have pushed a fix in #2773.

Mytherin added a commit that referenced this issue Dec 12, 2021
Fix #2761: correctly check that stderr is pointing to a terminal
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.

2 participants