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

[SNOW-1669514] Honor the Valuer/Stringer methods to resolve #1209 #1211

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

[SNOW-1669514] Honor the Valuer/Stringer methods to resolve #1209 #1211

wants to merge 15 commits into from

Commits on Sep 19, 2024

  1. try to recreate the issue with UUID to solve snowflakedb#1209

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    5a22458 View commit details
    Browse the repository at this point in the history

  2. [AB#1669514] fix UUID

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    cd958f0 View commit details
    Browse the repository at this point in the history

  3. [AB#1669514] remove commented code

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 19, 2024
    Configuration menu
    Copy the full SHA
    6d42602 View commit details
    Browse the repository at this point in the history

Commits on Sep 20, 2024

  1. [AB#1669514] alternate approach; more generalized

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    d4c0161 View commit details
    Browse the repository at this point in the history

  2. [AB#1669514] add stringer safe check; driver valuer check too

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    f84b52f View commit details
    Browse the repository at this point in the history

  3. [AB#1669514] Fix some comments

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    c925d7f View commit details
    Browse the repository at this point in the history

  4. [AB#1669514] simplify stringer check

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    97b609e View commit details
    Browse the repository at this point in the history

  5. [AB#1669514] simplify valuer

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    8f86d58 View commit details
    Browse the repository at this point in the history

  6. [AB#1669514] move the driver valuer short circuit above the other typ… 

    …e checks
    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    1454aca View commit details
    Browse the repository at this point in the history

  7. [AB#1669514] dont run stringer on everything arrays

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    1faf665 View commit details
    Browse the repository at this point in the history

  8. [AB#1669514] add array check too

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    9dac29c View commit details
    Browse the repository at this point in the history

  9. [AB#1669514] jsonFmtStr and ignore debug binary

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    28bf47e View commit details
    Browse the repository at this point in the history

  10. [AB#1669514] make it simpler

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    bb50506 View commit details
    Browse the repository at this point in the history

  11. [AB#1669514] revert that change; wont handle complex types

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    0d5b162 View commit details
    Browse the repository at this point in the history

Commits on Sep 23, 2024

  1. [AB#1669514] add test for SQL Null time

    @ChronosMasterOfAllTime
    ChronosMasterOfAllTime committed Sep 23, 2024
    Configuration menu
    Copy the full SHA
    e0b541f View commit details
    Browse the repository at this point in the history