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.

Sign up for GitHub

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

Jump to bottom

[go-client] support produce flush #3469

Merged
merged 16 commits into from
Jan 29, 2019
Merged

[go-client] support produce flush #3469

merged 16 commits into from
Jan 29, 2019

Conversation

wolfstudy
Copy link
Member

Fixes issue:3388

Motivation

support produce flush for go-client

wolfstudy and others added 14 commits January 24, 2019 11:45
@wolfstudy
[go-clint]add producer.GetLastSequenceID in Go client
8b2e106 
Signed-off-by: xiaolong.ran <ranxiaolong716@gmail.com>
@wolfstudy
fix comments and use LastSequenceID instead of GetLastSequenceID
472d0fd 
Signed-off-by: xiaolong.ran <ranxiaolong716@gmail.com>
@wolfstudy
fix consumer and producer test error
3193b07 
Signed-off-by: xiaolong.ran <ranxiaolong716@gmail.com>
@wolfstudy
[pulsar-client-go]provide setSequenceID interface
957cc82 
Signed-off-by: xiaolong.ran <ranxiaolong716@gmail.com>
@wolfstudy
add test case for setSequenceID
06829d7 
Signed-off-by: xiaolong.ran <ranxiaolong716@gmail.com>
@wolfstudy
rename ID -> SequenceID
100f2aa 
Signed-off-by: xiaolong.ran <ranxiaolong716@gmail.com>
@wolfstudy
support issue #3417
504e6b8 
Signed-off-by: xiaolong.ran <ranxiaolong716@gmail.com>
@wolfstudy
merge some changes
8e718f5 
Signed-off-by: xiaolong.ran <ranxiaolong716@gmail.com>
@wolfstudy
fix some issues about log and go mod
c73e3c5 
Signed-off-by: xiaolong.ran <ranxiaolong716@gmail.com>
@wolfstudy
Merge branch 'master' of github.com:apache/pulsar
5fe23b2
@wolfstudy
Merge branch 'master' of github.com:apache/pulsar
5a6ab11
@wolfstudy
support produce flush[issue#3388]
c88a16b 
Signed-off-by: xiaolong.ran <ranxiaolong716@gmail.com>
@wolfstudy
add produce flush
5ce58c7 
Signed-off-by: xiaolong.ran <ranxiaolong716@gmail.com>
@wolfstudy
Merge branch 'master' into master
a3df593
@merlimat merlimat added this to the 2.3.0 milestone Jan 29, 2019
merlimat
merlimat approved these changes Jan 29, 2019
View reviewed changes
Copy link
Contributor

@merlimat merlimat left a comment

Choose a reason for hiding this comment

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

👍

(It seems there's a format validation failure. You can fix C++ format with make format)

@wolfstudy
Copy link
Member Author

@merlimat yes, I am using the goland editor, the format may be different. Thank you for your help!

@wolfstudy
Copy link
Member Author

run cpp tests

@wolfstudy wolfstudy changed the title [go-client] suppor produce flush [go-client] support produce flush Jan 29, 2019
@wolfstudy
Copy link
Member Author

run cpp tests

@wolfstudy
Copy link
Member Author

run java8 tests

@merlimat merlimat merged commit 14d478d into apache:master Jan 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@merlimat merlimat merlimat approved these changes

Assignees
No one assigned
Labels
area/client
Projects
None yet
Milestone
2.3.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@wolfstudy @merlimat