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

Bulk copy metadata should not be cached for temporary tables #2585

Open
ulfeks opened this issue Jan 17, 2025 · 1 comment · May be fixed by #2594
Open

Bulk copy metadata should not be cached for temporary tables #2585

ulfeks opened this issue Jan 17, 2025 · 1 comment · May be fixed by #2594
Assignees
@machavan
Labels
Under Review Used for pull requests under review

Comments

@ulfeks
Copy link

ulfeks commented Jan 17, 2025

Driver version

12.8.1.jre11

SQL Server version

Microsoft SQL Server 2019 (RTM-CU19) (KB5023049) - 15.0.4298.1 (X64)
Jan 27 2023 16:44:09
Copyright (C) 2019 Microsoft Corporation Enterprise Edition:
Core-based Licensing (64-bit) on Windows Server 2022 Datacenter 10.0 (Build 20348: ) (Hypervisor)

Client Operating System

Microsoft Windows Server 2022 Datacenter

JAVA/JVM version

22.0.2

Table schema

N/A

Problem description

When using cacheBulkCopyMetadata=true, metadata about temporary tables are also cached. If the temporary tables have unique names the application will eventually run out of memory.

Expected behavior

Do not cache metadata for temporary tables, or make it optional.

Actual behavior

OutOfMemory Exception

Error message/stack trace

N/A

Any other details that can be helpful

N/A

JDBC trace logs

N/A
@github-project-automation github-project-automation bot moved this to To be triaged in MSSQL JDBC Jan 17, 2025
@machavan machavan self-assigned this Jan 17, 2025
@machavan machavan added the Under Investigation Used for issues under investigation label Jan 17, 2025
@lilgreenbird
Copy link
Contributor

hi @ulfeks

Thanks for reporting this we will investigate and work on a fix as soon as possible.

@machavan machavan added Under Review Used for pull requests under review and removed Under Investigation Used for issues under investigation labels Jan 28, 2025
@machavan machavan moved this from To be triaged to Under Peer Review in MSSQL JDBC Jan 28, 2025
@machavan machavan linked a pull request Jan 28, 2025 that will close this issue
Changed the scope of BULK_COPY_OPERATION_CACHE to connection. #2594
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@machavan machavan

Labels
Under Review Used for pull requests under review
Projects
MSSQL JDBC
Status: Under Peer Review
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Changed the scope of BULK_COPY_OPERATION_CACHE to connection. microsoft/mssql-jdbc
3 participants
@lilgreenbird @ulfeks @machavan