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

Implemented CLI task for ClickHouse #370

Merged
merged 11 commits into from
Sep 2, 2024
Merged

Implemented CLI task for ClickHouse #370

merged 11 commits into from
Sep 2, 2024
+158 −0

Conversation

iNikitaGricenko
Copy link
Contributor

@iNikitaGricenko iNikitaGricenko commented Aug 29, 2024 •
edited
Loading

What changes are being made and why?

How the changes have been QAed?

tasks:
  - id: query
    type: io.kestra.plugin.clickhouse.cli.ClickHouseCLI
    commands:
      - SELECT count() FROM s3('https://datasets-documentation.s3.eu-west-3.amazonaws.com/house_parquet/house_0.parquet')
     
  - id: query2
    type: io.kestra.plugin.clickhouse.cli.ClickHouseCLI
    commands:
      - SELECT * FROM system.tables

iNikitaGricenko and others added 10 commits August 16, 2024 21:56
@iNikitaGricenko
Added code, that allows users to pass table name, from which will be …
Loading
Loading status checks…
b885d94 
…retrieved columns names for insert
@iNikitaGricenko
Changed descriptions and titles. Added example to ClickHouse BulkInse…
Loading
Loading status checks…
fa86e55 
…rt. Added test to ClickHouseTest
@iNikitaGricenko
Fixed postgres tests. Added ssl parameters for new postgres tests
Loading
Loading status checks…
ee6e584
@loicmathieu
Update plugin-jdbc-vectorwise/src/test/java/io/kestra/plugin/jdbc/vec…
Loading
Loading status checks…
f4c9b00 
…torwise/BatchTest.java
@loicmathieu
Update plugin-jdbc-vectorwise/src/test/java/io/kestra/plugin/jdbc/vec…
Loading
Loading status checks…
90df9ca 
…torwise/BatchTest.java
@iNikitaGricenko
Merge branch 'kestra-io:master' into master
8530a80
@iNikitaGricenko
Added support for tsvector for postgres
Loading
Loading status checks…
c4b2945
@iNikitaGricenko
Merge branch 'kestra-io:master' into master
81426a9
@iNikitaGricenko
Implemented ClickHouse CLI task
49cdd82
@iNikitaGricenko
Merge branch 'kestra-io:master' into master
Loading
Loading status checks…
eea80c8
@iNikitaGricenko
Apply suggestions
Loading
Loading status checks…
e6af320
loicmathieu
loicmathieu approved these changes Aug 30, 2024
View reviewed changes
Copy link
Member

@loicmathieu loicmathieu left a comment

Choose a reason for hiding this comment

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

LGTM

@loicmathieu loicmathieu merged commit 674cd38 into kestra-io:master Sep 2, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@loicmathieu loicmathieu

Assignees
No one assigned
Labels
None yet
Projects
Pull Requests
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@iNikitaGricenko @loicmathieu