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.

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

chore: merge upstream #29

Merged
merged 46 commits into from
Dec 13, 2022
Merged

chore: merge upstream #29

merged 46 commits into from
Dec 13, 2022

Conversation

RainbowDashy
Copy link
Collaborator

cherry-pick every non-merge commit from github/gh-ost to bytebase/gh-ost

timvaillancourt and others added 30 commits December 13, 2022 17:49
* Merge pull request #31 from openark/zero-date

Support zero date and zero in date, via dedicated command line flag

* Merge pull request #32 from openark/existing-date-with-zero

Support tables with existing zero dates

* Remove un-needed ignore_versions file

* Fix new lint errors from golang-ci update

Co-authored-by: Shlomi Noach <[email protected]>
* Set transaction isolation in connections

* Revert load_map.go change

* Var rename

* Restore comment
)

* Use a transaction in applier `ReadMigrationRangeValues` func

* Private func names
* Add basic tests for applier

* Add header
* Add basic test for inspector

* Add header

* Fix return
table name is 'tbl' not 'tble'
@cla-bot
Copy link

cla-bot bot commented Dec 13, 2022

We require contributors to sign our Contributor License Agreement, and we don't have yours on file. In order for us to review and merge your code, please please sign CLA and add your name to contributors list.

go/logic/applier.go Outdated Show resolved Hide resolved
@cla-bot
Copy link

cla-bot bot commented Dec 13, 2022

We require contributors to sign our Contributor License Agreement, and we don't have yours on file. In order for us to review and merge your code, please please sign CLA and add your name to contributors list.

@d-bytebase d-bytebase merged commit 7ed4589 into main Dec 13, 2022
@RainbowDashy RainbowDashy deleted the main-merge-upstream branch December 14, 2022 02:39
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.

9 participants