Skip to content

KBC-513 update snflk driver to 2.22.5 #52

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
Open
wants to merge 228 commits into
base: main
Choose a base branch
from
Open

KBC-513 update snflk driver to 2.22.5 #52

wants to merge 228 commits into from

Conversation

zajca
Copy link
Member

@zajca zajca commented Mar 12, 2021 •
edited
Loading

zajca and others added 30 commits November 11, 2019 08:08
@zajca
Merge pull request #1 from keboola/vojta-azure-snowflake
a965b01 
Import files abs to snflk
@zajca
add export support
b8cf894
@zajca
add unit tests
f80f456
@zajca
test if ABS files extends BaseFile
bb6ee90
@zajca
test imported count
09f1323
@zajca
simplify interfaces with rowFetch from staging table
b021b96
@zajca
remove OVERWRITE = TRUE from export query
76d1066
@zajca
fix var name
1b13e99
@zajca
move count query to builder
d6d4789
@zajca
Merge pull request #2 from keboola/zajca-export-snflk-azure
c6ef0c9 
export from snflk to azure
@zajca
add readme and template
2bc801b
@zajca
Merge pull request #7 from keboola/zajca-readme
758c0de 
Add readme
@zajca
use csv options lib
c081032
@zajca
use vcs url with https
6e7f725
@zajca
remove keboola/php-component dependency
66d6c2e
@zajca
use public packagist package
f10a449
@zajca
downgrade keboola/php-csv-db-import 5.0.0->4.1.2 (bc in connection)
a930ef8
@zajca
Merge pull request #10 from keboola/zajca-csv-options
b370d8c 
Use CSVOptions lib
@zajca
add licence
4903900
@zajca
Merge pull request #11 from keboola/zajca-kbc-58
ee76aac 
KBC-58 Make package public on packagis
@zajca
fix php 7.1 compatibility
ff73309
@zajca
Merge pull request #13 from keboola/zajca-fix-php71
f19ca4c 
[fix] php 7.1 compatibility
@zajca
allow query bind
c3db5c7
@zajca
Merge pull request #14 from keboola/zajca-export-bind
c1f1e51 
Allow query bind
@zajca
sync version of keboola/php-csv-db-import with connection
e7a48ff
@zajca
Merge pull request #15 from keboola/zajca-sync-dep-version
7e284bb 
Sync version of keboola/php-csv-db-import with connection
@zajca
new SelectSource snowflake source
1ae83d6
@zajca
fix colon
d2e270f
@zajca
add new SqlSourceInterface, add SelectSourceTest
0c60ec8
@zajca
SqlSourceInterface adds new method getQueryBindings
56b4aaf
ErikZigo and others added 26 commits February 12, 2021 16:16
@ErikZigo
SnowflakeExportAdapterTest fixup
629d80c
@ErikZigo
Merge pull request #48 from keboola/erik-KBC-1285-snowflake-abs-unload
6d2147b 
Erik kbc 1285 snowflake abs unload
@zajca
skip casting on file->table imports in synapse
c7b1daf
@zajca
Merge pull request #43 from keboola/zajca-kbc-1250
117488d 
KBC-1250 skip casting on synapse imports
@zajca
update phpstan to 0.12
bcd6755
@zajca
add phpstan baseline
d6d9186
@zajca
SourceFile refactor internal methods
e9b5314
@zajca
add line endings detection for synapse imports
4dbcfd1
@zajca
update keboola/php-file-storage-utils
f7878a9
@zajca
add support for new line detection to SourceDirectory, fix empty mani…
801146a 
…fest use case
@zajca
add new Exceptions FileNotFoundException, ManifestNotFoundException, …
b72796a 
…use it in SourceFile, SourceDirectory
@zajca
add test testGetLineEndingSingleCrlf
44367a0
@zajca
Merge pull request #50 from keboola/zajca-kbc-1005
15d84d5 
KBC-1005 detect line ending in synapse import
@zajca
Merge pull request #49 from keboola/zajca-update-phpstan
a4beaa1 
update phpstan to 0.12
@zajca
refactoring and add stub data loading to S3
879015f
@zajca
create CopyCommandCsvOptionsHelper
570dad1
@zajca
add S3SourceTrait
eae01dd
@zajca
add S3 SourceFile
79b3063
@zajca
add S3 SourceDirectory
2730b6d
@zajca
add S3 SnowflakeImportAdapter
6b9e7a9
@zajca
update snowflake import tests for S3
c845b50
@zajca
add new test suites to run in CI
51bcfe6
@zajca
update phpstan baseline
7a70149
@zajca
add s3 variables to pipelines
4626f59
@zajca
update snflk driver to 2.22.5
b7e16a6
@zajca
update snflk driver GPG key
1ffe44c
Base automatically changed from zajca-kbc-513 to master March 12, 2021 14:03
tomasfejfar
tomasfejfar approved these changes Mar 13, 2021
View reviewed changes
Copy link
Contributor

@tomasfejfar tomasfejfar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Od verze 2.20 by mělo jít vypnout execuování queries v prepare (https://github.com/keboola/snowflake-odbc-debug/commit/149d688ba241b8fcdc4731676f0259ca10387d4b https://keboola.atlassian.net/browse/KBC-625). Nevím, jestli by nestálo za to to tady přidat, abysme zjistili jestli to funguje jak má :) Releasnuli to tady: https://webcache.googleusercontent.com/search?q=cache:zZsxfnmkWuMJ:https://community.snowflake.com/s/article/4-27-Release-Notes-August-3-6+&cd=1&hl=en&ct=clnk&gl=cz

Je taky otázka, jestli updatovat na latest verzi - oni ten release proces nemají úplně 100% takže ta latest je spíš taková beta občas :D (resp. tak to bylo dřív)

@zajca
Copy link
Member Author

zajca commented Mar 15, 2021 •
edited
Loading

Od verze 2.20 by mělo jít vypnout execuování queries v prepare (keboola/snowflake-odbc-debug@149d688 https://keboola.atlassian.net/browse/KBC-625). Nevím, jestli by nestálo za to to tady přidat, abysme zjistili jestli to funguje jak má :) Releasnuli to tady: https://webcache.googleusercontent.com/search?q=cache:zZsxfnmkWuMJ:https://community.snowflake.com/s/article/4-27-Release-Notes-August-3-6+&cd=1&hl=en&ct=clnk&gl=cz

Je taky otázka, jestli updatovat na latest verzi - oni ten release proces nemají úplně 100% takže ta latest je spíš taková beta občas :D (resp. tak to bylo dřív)

Nevím, ale podle čeho bys chtěl vybírat teda verzi driveru, jakože náhodné číslo? latest je BTW: latest je 2.30.0 ta byla ještě minulý týden v repu ale nebyla v changelogu a teď už není ani v repu :D je smazaná.
// taj minulý týden to byla 2.30.0 ale asi sa sekli teď je to 2.23.0 je v repu, ale ofiko neexistuje :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomasfejfar tomasfejfar tomasfejfar approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zajca @tomasfejfar @ErikZigo