You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cassandra.apache.org by Varun Barala <va...@gmail.com> on 2019/04/23 05:00:30 UTC

TimestampSerializer DEFAULKT_FORMATTER

Hi,

Commit:
https://github.com/apache/cassandra/commit/2635632270289fbbab2f3e3181f20aa98c34a879

Changes, as per the above commit, makes *DEFAULT_FORMATTER* minute level
precise however it was precise till seconds before this commit.

dateStringPatterns array has been updated but the used *index* in
*DEFAULT_FORMATTER* still remained the same(3).

https://github.com/apache/cassandra/commit/2635632270289fbbab2f3e3181f20aa98c34a879#diff-88e323487c81a7508ccbf79687229203R63

https://github.com/apache/cassandra/commit/2635632270289fbbab2f3e3181f20aa98c34a879#diff-88e323487c81a7508ccbf79687229203R32

Was it intentional? If it was not then please do let me know I'll create a
ticket and will submit the patch.

Thanks and regards,
Varun Barala

Re: TimestampSerializer DEFAULKT_FORMATTER

Posted by Varun Barala <va...@gmail.com>.
I found its fix:
https://github.com/apache/cassandra/commit/437bb9de77f54aa5a4a6a634ab3d2c753a17b3fc
though
it only fixed versions >=2.2.x

On Tue, Apr 23, 2019 at 1:00 PM Varun Barala <va...@gmail.com>
wrote:

> Hi,
>
> Commit:
> https://github.com/apache/cassandra/commit/2635632270289fbbab2f3e3181f20aa98c34a879
>
> Changes, as per the above commit, makes *DEFAULT_FORMATTER* minute level
> precise however it was precise till seconds before this commit.
>
> dateStringPatterns array has been updated but the used *index* in
> *DEFAULT_FORMATTER* still remained the same(3).
>
>
> https://github.com/apache/cassandra/commit/2635632270289fbbab2f3e3181f20aa98c34a879#diff-88e323487c81a7508ccbf79687229203R63
>
>
> https://github.com/apache/cassandra/commit/2635632270289fbbab2f3e3181f20aa98c34a879#diff-88e323487c81a7508ccbf79687229203R32
>
> Was it intentional? If it was not then please do let me know I'll create a
> ticket and will submit the patch.
>
> Thanks and regards,
> Varun Barala
>