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.

Sign up for GitHub

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

Jump to bottom

Incorrect response packet for Describe command while handling extended protocol requests #342

Closed
VWagen1989 opened this issue Jan 2, 2025 · 2 comments · Fixed by #352
Closed

Incorrect response packet for Describe command while handling extended protocol requests #342

VWagen1989 opened this issue Jan 2, 2025 · 2 comments · Fixed by #352
Labels
bug Something isn't working compatibility Be compatible with some old fashion request

Comments

@VWagen1989
Copy link
Contributor

Expected:
image
image

MyDuck produced:
image

It seems it's the NoData packet led to this problem.
@VWagen1989 VWagen1989 added bug Something isn't working compatibility Be compatible with some old fashion request labels Jan 2, 2025
@VWagen1989
Copy link
Contributor Author

VWagen1989 commented Jan 2, 2025 •
edited
Loading

Related to #340

@VWagen1989
Copy link
Contributor Author

Another unexptected result for an empty statement:
image
image

Expected:
image

VWagen1989 added a commit that referenced this issue Jan 6, 2025
@VWagen1989
fix: solve issue of handling extended query (#342)
18eada4
VWagen1989 added a commit that referenced this issue Jan 8, 2025
@VWagen1989
fix: solve issue of handling extended query (#342)
3cff241
@VWagen1989 VWagen1989 mentioned this issue Jan 8, 2025
Merged
VWagen1989 added a commit that referenced this issue Jan 9, 2025
@VWagen1989
fix: solve issue of handling extended query (#342)
47dc99f
VWagen1989 added a commit that referenced this issue Jan 9, 2025
@VWagen1989
fix: resolve compatibilities for Metabase (#340) (#352)
34b0213 
* fix: solve issue of handling extended query (#342)

* fix: add internal view pg_stat_user_tables and add query rewritten for type casting

* create internal macros to mimic some pg system functions

* wip: add pg_catalog.pg_get_indexdef

* fix: use a better regex pattern to replace the sys function names

* fix: make 'SET SESSION CHARACTERISTICS TRANSACTION ...' work and use session level pgtypes.Map for each session to encode results

* fix: cast DuckDB HUGEINT to pgtype.Numeric

* fix: adopt CR feedbacks

* fix: add tests and resolve failed tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working compatibility Be compatible with some old fashion request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: resolve compatibilities for Metabase (#340) apecloud/myduckserver
1 participant
@VWagen1989