NO-SNOW: Remove ALLOW_LARGE_LOBS_IN_EXTERNAL_SCAN from LobSizeLatestIT#2605
Merged
sfc-gh-dheyman merged 1 commit intoApr 29, 2026
Merged
Conversation
Collaborator
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Contributor
There was a problem hiding this comment.
Pull request overview
Removes use of the ALLOW_LARGE_LOBS_IN_EXTERNAL_SCAN session parameter from the LobSizeLatestIT integration test, avoiding reliance on a session setting that is no longer needed/available for the PUT/COPY/GET LOB-size coverage.
Changes:
- Deleted conditional session alteration enabling
ALLOW_LARGE_LOBS_IN_EXTERNAL_SCANfor larger LOB sizes intestPutAndGet.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
sfc-gh-merbel
approved these changes
Apr 29, 2026
sfc-gh-rkowalski
approved these changes
Apr 29, 2026
sfc-gh-dheyman
deleted the
dheyman/fix-lob-size-allow-large-lobs-removed
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TL;DR
Removed the session parameter override for
ALLOW_LARGE_LOBS_IN_EXTERNAL_SCANin the LOB size integration test, because tests started failing withSQL compilation error: invalid parameter 'ALLOW_LARGE_LOBS_IN_EXTERNAL_SCANWhat changed?
The conditional statement that executed
alter session set ALLOW_LARGE_LOBS_IN_EXTERNAL_SCAN = truefor LOB sizes exceeding the original limit (16MB–128MB) has been removed from thetestPutAndGettest method.How to test?
Run the
LobSizeLatestITintegration tests across the various LOB size and result format combinations to confirm that large LOB PUT/GET operations succeed without requiring the session parameter to be explicitly set.Why make this change?
The
ALLOW_LARGE_LOBS_IN_EXTERNAL_SCANsession parameter has been removed from Snowflake (server release 10.15). Since early-access accounts have started rolling forward to 10.15, JDBC CI began failing.