Skip to content

Comments

MOLT Verify transformations and filter rules#22700

Open
taroface wants to merge 4 commits intomainfrom
molt-verify-transformations
Open

MOLT Verify transformations and filter rules#22700
taroface wants to merge 4 commits intomainfrom
molt-verify-transformations

Conversation

@taroface
Copy link
Contributor

@netlify
Copy link

netlify bot commented Feb 18, 2026

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 23d3c32
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/699e0daa88300700088326f4

@netlify
Copy link

netlify bot commented Feb 18, 2026

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 23d3c32
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/699e0daae6c93000083f0eec

@github-actions
Copy link

Files changed:

@netlify
Copy link

netlify bot commented Feb 18, 2026

Netlify Preview

Name Link
🔨 Latest commit 23d3c32
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/699e0daabaf8220008db3b74
😎 Deploy Preview https://deploy-preview-22700--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

@KeithCh KeithCh left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@bsanchez-the-roach bsanchez-the-roach left a comment

Choose a reason for hiding this comment

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

Some comments are minor nits about phrasing that you can take or leave, others are slightly more substantial.


### Verify transformed data

If you applied [transformations during `molt fetch`]({% link molt/molt-fetch.md %}#transformations), you can apply the same transformations with MOLT Verify to match source data with the transformed target data.
Copy link
Contributor

Choose a reason for hiding this comment

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

When do you write it as MOLT Fetch vs. molt fetch

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I generally use the former when referring generally to the product, and the latter when describing the running process.

## Known limitations

- MOLT Verify compares 20,000 rows at a time by default, and row values can change between batches, potentially resulting in temporary inconsistencies in data. To configure the row batch size, use the `--row_batch_size` [flag](#flags).
- MOLT Verify only supports comparing one MySQL database to a whole CockroachDB schema (which is assumed to be `public`).
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm confused by this. Does this not contradict the fact that MySQL sources are supported for selective data verification?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was written a long time ago and I think it can be removed. I believe what it's trying to say is that MySQL doesn't have the schema concept from PG, so MySQL databases map to CRDB public schemas by default. (We make this point in e.g. Load and Replicate.) This isn't really a known limitation IMO, so I'll make the point inline under the MySQL connection string example.

taroface and others added 3 commits February 24, 2026 15:04
Co-authored-by: bsanchez-the-roach <brandon.sanchez@cockroachlabs.com>
Co-authored-by: bsanchez-the-roach <brandon.sanchez@cockroachlabs.com>
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.

4 participants