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

Use vendor #80

Merged
merged 2 commits into from
Apr 19, 2016
Merged

Use vendor #80

merged 2 commits into from
Apr 19, 2016

Conversation

siddontang
Copy link
Contributor

Etcd and gRPC updates too frequently, and ci/docker build may fail sometimes, so we should use vendor to guarantee compile ok.

@ngaut @qiuyesuifeng

@ngaut
Copy link
Member

ngaut commented Apr 19, 2016

LGTM

1 similar comment
@qiuyesuifeng
Copy link
Contributor

LGTM

@qiuyesuifeng qiuyesuifeng merged commit 77305f1 into master Apr 19, 2016
@qiuyesuifeng qiuyesuifeng deleted the siddontang/vendor branch April 19, 2016 14:44
rleungx added a commit to rleungx/pd that referenced this pull request Jun 5, 2023
* mcs, tso: fix ts fallback caused by multi-primary of the same keyspace group  (tikv#6362)

* Change participant election-prifix from listen-addr to advertise-listen-addr to gurantee uniqueness.

Signed-off-by: Bin Shi <binshi.bing@gmail.com>

* Add TestPariticipantStartWithAdvertiseListenAddr

Signed-off-by: Bin Shi <binshi.bing@gmail.com>

* Add comments to fix go fmt errors

Signed-off-by: Bin Shi <binshi.bing@gmail.com>

---------

Signed-off-by: Bin Shi <binshi.bing@gmail.com>
Co-authored-by: Ryan Leung <rleungx@gmail.com>

* keyspace, tso: fix next revision to watch after watch/Get/RangeScan (tikv#6353)

ref tikv#6232

The next revision to watch should always be Header.Revision + 1 where header is response header of watch/Get/RangeScan

Signed-off-by: Bin Shi <binshi.bing@gmail.com>

---------

Signed-off-by: Bin Shi <binshi.bing@gmail.com>
Co-authored-by: Ryan Leung <rleungx@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