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

DatabaseMetaData#getProcedures not ordered as specified. #2321

Open
onacit opened this issue Feb 5, 2024 · 2 comments
Open

DatabaseMetaData#getProcedures not ordered as specified. #2321

onacit opened this issue Feb 5, 2024 · 2 comments
Labels
Backlog The topic in question has been recognized and added to development backlog
Projects

Comments

@onacit
Copy link

onacit commented Feb 5, 2024

Driver version

12.6.0.jre11

SQL Server version

Maven home: /opt/homebrew/Cellar/maven/3.9.6/libexec
Java version: 21.0.2, vendor: Eclipse Adoptium, runtime: /Library/Java/JavaVirtualMachines/temurin-21.jdk/Contents/Home
Default locale: en_KR, platform encoding: UTF-8
OS name: "mac os x", version: "14.3", arch: "aarch64", family: "mac"

Client Operating System

Maven home: /opt/homebrew/Cellar/maven/3.9.6/libexec
Java version: 21.0.2, vendor: Eclipse Adoptium, runtime: /Library/Java/JavaVirtualMachines/temurin-21.jdk/Contents/Home
Default locale: en_KR, platform encoding: UTF-8
OS name: "mac os x", version: "14.3", arch: "aarch64", family: "mac"

JAVA/JVM version

Table schema

?

Problem description

getProcedures's result should be ordered as specified yet I got,

group is not sorted according to Comparator$$Lambda/0x00000008000fe3d0 comparator because element 202:
  Procedure(super=AbstractMetadataType(super=com.github.jinahya.database.metadata.bind.Procedure@bd5520b1, unmappedValues={NUM_INPUT_PARAMS=-1, NUM_OUTPUT_PARAMS=-1, NUM_RESULT_SETS=-1}),
    procedureCat=master,
    procedureSchem=sys,
    procedureName=sp_assemblies_rowset_rmt;1,
    remarks=null,
    procedureType=2,
    specificName=null)
is not less or equal than element 203:
  Procedure(super=AbstractMetadataType(super=com.github.jinahya.database.metadata.bind.Procedure@bd5520b1, unmappedValues={NUM_INPUT_PARAMS=-1, NUM_OUTPUT_PARAMS=-1, NUM_RESULT_SETS=-1}),
    procedureCat=master,
    procedureSchem=sys,
    procedureName=sp_assemblies_rowset2;1,
    remarks=null, procedureType=2,
    specificName=null)

Expected behavior

Actual behavior

Error message/stack trace

Any other details that can be helpful

JDBC trace logs

@Jeffery-Wasty
Copy link
Member

It's not clear at the moment how much can be done from the driver's side. We'll look into this and see a) can the ordering be done from the JDBC side, and b) the potential impact of changing how we order. We'll update this issue when we have more information.

@Jeffery-Wasty Jeffery-Wasty added this to Under Investigation in MSSQL JDBC via automation Feb 5, 2024
@Jeffery-Wasty
Copy link
Member

This will be added to our backlog. Putting aside whether this is possible (still unclear), a change would involve an extra, substantial, performance impact, and we're still not sure what the downstream effect of changing the order from what SQL server gives us is. We'll revisit this during a later triage.

@Jeffery-Wasty Jeffery-Wasty added the Backlog The topic in question has been recognized and added to development backlog label Feb 7, 2024
@Jeffery-Wasty Jeffery-Wasty moved this from Under Investigation to Backlog in MSSQL JDBC Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backlog The topic in question has been recognized and added to development backlog
Projects
MSSQL JDBC
  
Backlog
Development

No branches or pull requests

2 participants