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

Support client name and version report #3338

Closed
yangbodong22011 opened this issue Mar 30, 2023 · 0 comments · Fixed by #3356
Closed

Support client name and version report #3338

yangbodong22011 opened this issue Mar 30, 2023 · 0 comments · Fixed by #3356
Assignees
@yangbodong22011

Comments

@yangbodong22011
Copy link
Collaborator

refer: redis/redis#11758
@yangbodong22011 yangbodong22011 self-assigned this Mar 30, 2023
yangbodong22011 added a commit to yangbodong22011/jedis that referenced this issue Apr 4, 2023
@yangbodong22011
Support report libname and libver to Redis
b1b17bd 
redis#3338
@yangbodong22011 yangbodong22011 mentioned this issue Apr 4, 2023
Merged
yangbodong22011 added a commit to yangbodong22011/jedis that referenced this issue Apr 6, 2023
@yangbodong22011 @sazzad16
Support report libname and libver to Redis
29f7c60 
redis#3338
Co-authored-by: M Sazzadul Hoque <7600764+sazzad16@users.noreply.github.com>
yangbodong22011 added a commit to yangbodong22011/jedis that referenced this issue Apr 7, 2023
@yangbodong22011 @sazzad16
Support report libname and libver to Redis
406189c 
redis#3338
Co-authored-by: M Sazzadul Hoque <7600764+sazzad16@users.noreply.github.com>
sazzad16 added a commit that referenced this issue Apr 7, 2023
@yangbodong22011 @sazzad16
Support report libname and libver to Redis (#3356)
5192937 
* Support report libname and libver to Redis

#3338
Co-authored-by: M Sazzadul Hoque <7600764+sazzad16@users.noreply.github.com>

* Refactor ClientAttributeOption

Co-authored-by: M Sazzadul Hoque <7600764+sazzad16@users.noreply.github.com>

* Apply suggestions from code review

* Keep select()

* Update src/main/java/redis/clients/jedis/Connection.java

Co-authored-by: M Sazzadul Hoque <7600764+sazzad16@users.noreply.github.com>

---------

Co-authored-by: M Sazzadul Hoque <7600764+sazzad16@users.noreply.github.com>
@sazzad16 sazzad16 mentioned this issue Apr 10, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yangbodong22011 yangbodong22011

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support report libname and libver to Redis yangbodong22011/jedis
1 participant
@yangbodong22011