8000 python scripts to run checksums by aadant · Pull Request #80 · Altinity/clickhouse-sink-connector · GitHub
[go: up one dir, main page]

Skip to content

python scripts to run checksums - #80

Merged
subkanthi merged 3 commits into
Altinity:developfrom
aadant:table_checksums
Sep 13, 2022
Merged

subkanthi merged 3 commits into
Altinity:developfrom
aadant:table_checksums

Conversation

@aadant
@aadant aadant commented Sep 12, 2022 •
Copy link
Copy Markdown
Collaborator

Python code to compute checksums between MySQL and CH

@subkanthi
subkanthi changed the base branch from main to develop September 13, 2022 17:20
return (query, select, order_by_columns, external_column_types)


def selectTableStatements(table, query, select_query, order_by, external_column_types):
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

can we please change this to snake case, so its consistent.

return res


def getTableChecksumQuery(table):
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

can we please change this to snake case, so its consistent.

return (rowset, rowcount)


def executeStatement(strSql):
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

can we please change this to snake case, so its consistent.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

fill fix this in a separate PR.

@subkanthi
subkanthi merged commit 09a228a into Altinity:develop Sep 13, 2022
@subkanthi subkanthi added this to the Release 0.3.0 milestone Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0