-
Notifications
You must be signed in to change notification settings - Fork 201
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
Support for datetime2 data type #751
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can see many reformatting changes because your IDE is configured differently for rewritten (type == String.class) as (type==String.class)
can you remove those un-necessary changes ?
My IDE settings are correct, those lines were re-formatted because IDEA formats changed method bodies. It's just a few of them in a couple of methods. |
@vietj anything else beyond formatting? |
@tsegismont you seem to have peculiar IDE settings. Most vertx code base uses |
80317af
to
832b532
Compare
@vietj I use IDEA with default settings + the plugin for Anything else beyond formatting? |
can you rebase @tsegismont ? |
@vietj just rebased on |
@@ -207,6 +208,8 @@ private void encodeParamValue(ByteBuf payload, Object value) { | |||
encodeDateNParameter(payload, (LocalDate) value); | |||
} else if (value instanceof LocalTime) { | |||
encodeTimeNParameter(payload, (LocalTime) value, (byte) 6); | |||
} else if (value instanceof LocalDateTime) { | |||
encodeDateTimeNParameter(payload, (LocalDateTime) value, (byte) 6); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any opinions about the scale? I'm not sure if I ever got it right to hardcode it as 6 to not truncating any nano seconds in the LocalTime
parameter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I filed #752 to make sure we analyze and take any action needed.
See eclipse-vertx#608 Signed-off-by: Thomas Segismont <[email protected]>
See #608