-
Notifications
You must be signed in to change notification settings - Fork 24
Comparing changes
Open a pull request
base repository: cloudquery/plugin-sdk
base: v4.7.1-rc1
head repository: cloudquery/plugin-sdk
compare: v4.8.0-rc1
- 9 commits
- 22 files changed
- 4 contributors
Commits on Jul 4, 2023
-
fix(naming): Rename
SyncMessages.InsertMessage()to `SyncMessages.G……etInserts()` (#1070) Also add `SyncInserts.GetRecords()` for parity with `WriteInserts`
Configuration menu - View commit details
-
Copy full SHA for ab9e768 - Browse repository at this point
Copy the full SHA ab9e768View commit details -
fix(testdata): Exclude only the correct type (#1067)
`c.Type.ID()` is same for all Arrow extension types so instead of excluding the JSON type from list testdata, it excludes all extensions.
Configuration menu - View commit details
-
Copy full SHA for 1c72fb2 - Browse repository at this point
Copy the full SHA 1c72fb2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 888c9ee - Browse repository at this point
Copy the full SHA 888c9eeView commit details
Commits on Jul 5, 2023
-
fix: Deterministic ordering for records returned by readAll in tests (#…
…1072) This: - adds an `id` field that is used as an auto-incrementing integer field in tests, and can be used to order records by so that tests can pass regardless of sort order returned by a plugin's Read method - makes null rows an explicit option (rather than every other row being null) and starts using null rows in tests again
Configuration menu - View commit details
-
Copy full SHA for cf7510f - Browse repository at this point
Copy the full SHA cf7510fView commit details -
feat(transformers): Add
Applyto apply extra transformations (#1069)// Apply applies the given transformers to the given Tables, and recursively on the relations. // This is useful for applying transformers that are not defined in the table definitions. To apply the table-definition transformers, use TransformTables.
Configuration menu - View commit details
-
Copy full SHA for a40598e - Browse repository at this point
Copy the full SHA a40598eView commit details -
fix: Reverse order of records in memdb (#1075)
Shuffling missed a case (I think) because we were using the same seed every time. Iterating over the items in reverse should have the same effect and is still deterministic. Same fix as in #1074 but also adds a test for it.
Configuration menu - View commit details
-
Copy full SHA for 8356590 - Browse repository at this point
Copy the full SHA 8356590View commit details -
Configuration menu - View commit details
-
Copy full SHA for 88f08ee - Browse repository at this point
Copy the full SHA 88f08eeView commit details -
fix: Reset timers on flush (#1076)
This updates the ticker logic in the batch writers to reset the ticker when a flush happens. This is better, as it still guarantees that a message won't be delayed by more than batch_timeout, but we don't risk flushing a very small batch because we must flush at regular intervals either. The choice of resetting _after_ the flush is deliberate: it means that the maximum amount of time between flushes is: ``` max_time = flush_time + batch_timeout ``` otherwise we could do: ``` max_time = batch_timeout ``` but if a flush were to then longer than the batch timeout, we can end up in a cycle of flushing again immediately after the previous flush finishes.
Configuration menu - View commit details
-
Copy full SHA for 767327f - Browse repository at this point
Copy the full SHA 767327fView commit details -
chore(main): Release v4.8.0-rc1 (#1071)
🤖 I have created a release *beep* *boop* --- ## [4.8.0-rc1](v4.7.1-rc1...v4.8.0-rc1) (2023-07-05) ### Features * **transformers:** Add `Apply` to apply extra transformations ([#1069](#1069)) ([a40598e](a40598e)) ### Bug Fixes * Deterministic ordering for records returned by readAll in tests ([#1072](#1072)) ([cf7510f](cf7510f)) * Handle null-related test options ([#1074](#1074)) ([88f08ee](88f08ee)) * **naming:** Rename `SyncMessages.InsertMessage()` to `SyncMessages.GetInserts()` ([#1070](#1070)) ([ab9e768](ab9e768)) * Reset timers on flush ([#1076](#1076)) ([767327f](767327f)) * Reverse order of records in memdb ([#1075](#1075)) ([8356590](8356590)) * **scalar:** Test `AppendTime` on TimestampBuilder ([#1068](#1068)) ([888c9ee](888c9ee)) * **testdata:** Exclude only the correct type ([#1067](#1067)) ([1c72fb2](1c72fb2)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Configuration menu - View commit details
-
Copy full SHA for ec4d473 - Browse repository at this point
Copy the full SHA ec4d473View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v4.7.1-rc1...v4.8.0-rc1