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
Incorrect implementation of tuple factory method #400
Comments
Tuple built from List method does not work |
why six elements.... if elements size more than six? |
The intention of this method is to copy all elements in the List to a new
Tuple. Both the documentation and code need to be updated.
shenzhou-6 <notifications@github.com> 于 2019年8月24日周六 02:00写道:
… why six elements.... if elements size more than six?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#400?email_source=notifications&email_token=ABJV4JLP2JRLZRWPXOV5W7TQGAQSXA5CNFSM4IO7JW6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5A5HDI#issuecomment-524407693>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABJV4JOX5VBMO4M57C6VEL3QGAQSXANCNFSM4IO7JW6A>
.
|
yes this is a bug, in the mean time people can use the new Tuple.wrap(list) method |
BillyYccc
pushed a commit
to BillyYccc/vertx-sql-client
that referenced
this issue
Aug 30, 2019
vietj
added a commit
that referenced
this issue
Sep 9, 2019
jeoffreylim
pushed a commit
to jeoffreylim/vertx-sql-client
that referenced
this issue
Sep 12, 2019
…x#400 Signed-off-by: Jeoffrey Lim <jeoffreyl@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
vertx-sql-client/vertx-sql-client/src/main/java/io/vertx/sqlclient/Tuple.java
Lines 197 to 206 in e8ef0be
The text was updated successfully, but these errors were encountered: