call _compiler_dispatch for merge_into and copy_into clauses #372
Conversation
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
|
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
* feat: Add temporary option to the create stage command. * update changelog * Update tests/test_create.py Co-authored-by: Adam Ling <[email protected]> --------- Co-authored-by: DanCardin <[email protected]> Co-authored-by: Adam Ling <[email protected]>
* Bump up version to 1.4.8 * update version * update release version to 1.5.0
…nowflakedb#446) * SNOW-892284: Fix boolean parameter parsing from URL query * Fix cache_column_metadata being parsed as connector argument cache_column_metadata is a snowflake-sqlalchemy argument, not a snowflake-connector-python argument so it should be set and ommitted from the arguments list before the call to the connector is made. * Add link to Github issue for dealing with other URL param types * Add patch note * Handle case when cache_column_metadata isn't set * Load cache_column_metadata from query * Add fallthrough for unknown parameters * Fix lint * Update DESCRIPTION.md * Add more inline documentation --------- Co-authored-by: Saulius Beinorius <[email protected]>
sfc-gh-dszmolka
requested review from
sfc-gh-aling,
sfc-gh-aalam,
sfc-gh-astus and
sfc-gh-mraba
* SNOW-1065172-gh-workflow-optimization: workflow optimisation --------- Co-authored-by: Adam Stus <[email protected]> Co-authored-by: Tomasz Urbaszek <[email protected]>
…g repository name (snowflakedb#476)
…nv to metadata (snowflakedb#486) * SNOW-1324105-dependency-pinning: move versions pinning from project env to metadata * SNOW-1324105-dependency-pinning: update version and description * SNOW-1324105-dependency-pinning: remove duplicated command in CI job
* Fix typo in README.md --------- Co-authored-by: Norman Rosner <[email protected]> Co-authored-by: Anthony Holten <[email protected]>
* Fix import BOOLEAN error * Update imports * Add test for explicit imports
Improve index reflection
* SNOW-1776332 Add support for OBJECT * Updated description.md * Add missing @pytest.mark.requires_external_volume * Tuple validation in OBJECT class
* Add support for autocommit
* Fix return value of snowflake get_table_names --------- Co-authored-by: T Pham <[email protected]>
…nowflakedb#545) * Changed default behavior of SnowflakeDialect to disable the use of / division operator as floor div. Changed flag div_is_floor_div to False. * Update Description.md * Added flag to allow customer to test new behavior of div_is_floordiv that will be introduce, using new flag force_div_floordiv allow to test the new division behavior. Update sa14:scripts to ignore feature_v20 from execution * Added warning for use of div_is_floor_div with `True` value. Added tests to validate results of true and floor divisions using `force_div_is_floordiv` flag.
* Fix drop support for SA array * Update DESCRIPTION.md
In maintenance mode, showing it in readme file
…lumns. (snowflakedb#586) Trying to get all columns generates errors when parameters enable extra columns for describe table command.
Fixing exception caused by dependency on number of columns returned by DESCRIBE TABLE
* Fix failing tests * Try fixing again * Fix test third try * Fix tests forth try, add insecure_mode instead with disable_ocsp_checks * Remove skip, it is not necessary
…#592) * Fix issue * Update release notes * Add tests
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Please answer these questions before submitting your pull requests. Thanks!
What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes SNOW-723663: MergeInto and CopyInto do not respect schema_translate_map #371
Fill out the following pre-review checklist:
Please describe how your code solves the related issue.
This change passes the
MergeInto - target, source, onandCopyInto.intovalues to the compiler