-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Improvement of ResultSet metadata #27321
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
base: master
Are you sure you want to change the base?
Conversation
b2f3ef3 to
f89090c
Compare
client/trino-jdbc/src/main/java/io/trino/jdbc/TrinoDatabaseMetaData.java
Outdated
Show resolved
Hide resolved
core/trino-main/src/main/java/io/trino/execution/QueryStateMachine.java
Outdated
Show resolved
Hide resolved
client/trino-jdbc/src/test/java/io/trino/jdbc/TestTrinoDatabaseMetaData.java
Outdated
Show resolved
Hide resolved
8583dab to
9bf1d1e
Compare
3d52419 to
43eefdd
Compare
core/trino-main/src/main/java/io/trino/execution/ColumnInfo.java
Outdated
Show resolved
Hide resolved
core/trino-main/src/main/java/io/trino/execution/ColumnInfo.java
Outdated
Show resolved
Hide resolved
core/trino-main/src/main/java/io/trino/execution/QueryExecution.java
Outdated
Show resolved
Hide resolved
71e76fa to
f28ac38
Compare
|
Perhaps it would be a good idea to immediately add the missing information that will allow LibreOffice Base to efficiently update ResultSets, namely:
This will allow Base to recognize auto-incremented columns and allow me to know which ones have a calculated value. |
19b1944 to
01c4446
Compare
|
@findepi |
bc08bf9 to
0d1409d
Compare
I am not ready for this topic. Would prefer for that to be separate PR. |
71e76fa to
3baf4a3
Compare
|
@findepi, Okay, I'm reverted and will propose a PR just for that. |
Description
These changes add the following to the metadata of a ResultSet:
They are intended to address issue #22306
Additional context and related issues
Release notes
( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text: