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

ci: build on both Ubuntu and macOS #2

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

overvenus
Copy link
Owner

What problem does this PR solve?

Add CI job for building BR on both Ubuntu and macOS.

Check List

Tests

  • Unit test

Related changes

  • Need to cherry-pick to the release branch

Release note

  • No release note

3pointer and others added 7 commits January 31, 2021 09:13
* refactor ingester

* adapt ingester

* fmt & add some comments

* fix hound

* fix hound

* fix

* fix ci

* fix test

* fix test

* fix ci

* fix lint

* fix lint

* fix index panic

* sleep more time to wait ticdc synced

* address comment

* address comment

* fix test

* fix resolve lock

* fix hound

* fix nextkey

* fix hound

* fix lint

* fix ci

* refactor log

* fix deadlock

* change concurrency config

* fix hound

* fix remain range

* fix hound

* fix sort bug

* fix infinte loop

* fix test: resolve ts is 0

* update ingester coverage

* add seek for iterator & add some unit test

* address comment

* remove concurrency iter

* fix

* fix lost

* change range to [start, end)

* add iter producer

* fix build

* remove iter's lock

* fix build

* Update pkg/kv/kv.go

Co-authored-by: glorv <glorvs@163.com>

* address comment

* Update pkg/kv/kv.go

Co-authored-by: glorv <glorvs@163.com>

* address comment

* Update pkg/kv/kv.go

Co-authored-by: kennytm <kennytm@gmail.com>

* Update pkg/restore/split_client.go

Co-authored-by: kennytm <kennytm@gmail.com>

* adapt logutil

* address comment

* fix iter.Next()

* fix build

Co-authored-by: glorv <glorvs@163.com>
Co-authored-by: kennytm <kennytm@gmail.com>
Signed-off-by: Neil Shen <overvenus@gmail.com>
Signed-off-by: Neil Shen <overvenus@gmail.com>
Signed-off-by: Neil Shen <overvenus@gmail.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.

3 participants