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

Test pr 1211 #1269

Closed
wants to merge 47 commits into from
Closed

Test pr 1211 #1269

wants to merge 47 commits into from

Conversation

sfc-gh-pfus
Copy link
Collaborator

Description

SNOW-XXX Please explain the changes you made here.

Checklist

  • Created tests which fail without the change (if possible)
  • Extended the README / documentation, if necessary

@ChronosMasterOfAllTime
try to recreate the issue with UUID to solve #1209
d3aaee0
@ChronosMasterOfAllTime
[AB#1669514] fix UUID
478b593
@ChronosMasterOfAllTime
[AB#1669514] remove commented code
b76c390
@ChronosMasterOfAllTime
[AB#1669514] alternate approach; more generalized
018095b
@ChronosMasterOfAllTime
[AB#1669514] add stringer safe check; driver valuer check too
f5808c4
@ChronosMasterOfAllTime
[AB#1669514] Fix some comments
cd4468a
@ChronosMasterOfAllTime
[AB#1669514] simplify stringer check
38b21cd
@ChronosMasterOfAllTime
[AB#1669514] simplify valuer
e81bbf3
@ChronosMasterOfAllTime
[AB#1669514] move the driver valuer short circuit above the other typ…
8eb0367 
…e checks
@ChronosMasterOfAllTime
[AB#1669514] dont run stringer on everything arrays
c3303c5
@ChronosMasterOfAllTime
[AB#1669514] add array check too
14bc235
@ChronosMasterOfAllTime
[AB#1669514] jsonFmtStr and ignore debug binary
22af12d
@ChronosMasterOfAllTime
[AB#1669514] make it simpler
9d00074
@ChronosMasterOfAllTime
[AB#1669514] revert that change; wont handle complex types
15d3892
@ChronosMasterOfAllTime
[AB#1669514] add test for SQL Null time
ab8cfff
@ChronosMasterOfAllTime
[AB#1669514] make it simpler and mayyybe easier to understand
1b28c16
@ChronosMasterOfAllTime
[AB#1669514] even simpler
fe4f737
@ChronosMasterOfAllTime
[AB#1669514] nil safety on err
28e3e72
@ChronosMasterOfAllTime
[AB#1669514] more complete converter checks
aa8f8c2
@ChronosMasterOfAllTime
[AB#1669514] structured read test
97e988f
@ChronosMasterOfAllTime
[AB#1669514] structured r/w tests
8458787
@ChronosMasterOfAllTime
[AB#1669514] driver test?
532d2f7
@ChronosMasterOfAllTime
[AB#1669514] fix driver test
bdbab90
@ChronosMasterOfAllTime
[AB#1669514] not quite right, but structured driver test is almost done
393eb56
@ChronosMasterOfAllTime
[AB#1669514] no hard dependencies on googleUUID
e7e805c
@ChronosMasterOfAllTime
[AB#1669514] move type to test file so it's not bundled
dd80100
@ChronosMasterOfAllTime
[AB#1669514] dont commit the commented query too large to print error
bf927a4
@ChronosMasterOfAllTime
[AB#1669514] debug mode for local help
16d2c23
@ChronosMasterOfAllTime
[AB#1669514] bump checkout; fix typo in yaml
5a71ba0
@ChronosMasterOfAllTime
[AB#1669514] Notes in bind_uploader
cf0c850
ChronosMasterOfAllTime and others added 17 commits December 9, 2024 07:59
@ChronosMasterOfAllTime
[AB#1669514] data.Message fix
2742ee7
@ChronosMasterOfAllTime
[AB#1669514] revert atoierr
d3b224a
@ChronosMasterOfAllTime
[AB#1669514] JSON test vs basic type
dd72c7c
@ChronosMasterOfAllTime
[AB#1669514] arrow test
a15a02a
@ChronosMasterOfAllTime
[AB#1669514] fix array binding
c2dd9c0
@ChronosMasterOfAllTime
[AB#1669514] make it slightly faster by using nlogn complexity
a1137f8
@ChronosMasterOfAllTime
back to 100K
9f4877c
@ChronosMasterOfAllTime
[AB#1669514] simplify if check in test
b14e863
@ChronosMasterOfAllTime
[AB#1669514] More performant sort
a9416a5
@ChronosMasterOfAllTime
[AB#1669514] use internal assert method
f829845
@ChronosMasterOfAllTime
[AB#1669514] fix errors; tests will still fail
f829ed9
@ChronosMasterOfAllTime
[AB#1669514] type assertions on casts in tests to avoid panics
e508b9e
@sfc-gh-pfus @ChronosMasterOfAllTime
[AB#1669514] Fixes
ae1fcab
@ChronosMasterOfAllTime
[AB#1669514] move json transform check to function to keep it in a si…
0a002c8 
…ngle place
@ChronosMasterOfAllTime
[AB#1669514] add to readme and update it for debug mode
e89206f
@ChronosMasterOfAllTime
[AB#1669514] Linter fix
3830f56
@ChronosMasterOfAllTime
[AB#1669514] whitespace formats
b07b709
@sfc-gh-pfus sfc-gh-pfus deleted the test_pr_1211 branch December 10, 2024 14:02
@github-actions github-actions bot locked and limited conversation to collaborators Dec 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sfc-gh-pfus @ChronosMasterOfAllTime