[ 
https://issues.apache.org/jira/browse/CASSANDRA-15257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17024822#comment-17024822
 ] 

Dinesh Joshi commented on CASSANDRA-15257:
------------------------------------------

Hi [~brandon.williams], this patch breaks {{TestCqlsh::test_datetime_values}}. 
CI run 
[here|https://circleci.com/gh/dineshjoshi/cassandra/1565#tests/containers/40]. 
Here are the errors.
{noformat}
E       AssertionError: Failed to execute cqlsh: <stdin>:13:InvalidRequest: 
Error from server: code=2200 [Invalid query] message="Unable to coerce 
'1582-1-1' to a formatted date (long)"
E         <stdin>:15:InvalidRequest: Error from server: code=2200 [Invalid 
query] message="Unable to coerce '0-1-1' to a formatted date (long)"
E         <stdin>:17:InvalidRequest: Error from server: code=2200 [Invalid 
query] message="Unable to coerce '1-1-1' to a formatted date (long)"
E         <stdin>:19:InvalidRequest: Error from server: code=2200 [Invalid 
query] message="Unable to coerce '9999-1-1' to a formatted date (long)"
E         <stdin>:21:InvalidRequest: Error from server: code=2200 [Invalid 
query] message="Unable to coerce '10000-1-1' to a formatted date (long)"
{noformat}

This test passes at commit 9a280516ca8b9e730ae0648e5e29ee6280605132 and breaks 
on trunk (2f2b2ae607b2531836c805da913ca2e545162275).

I am not sure if the intention of this ticket was to simply eliminate the Joda 
Time dependency while maintaining backward compatibility. If that is the 
intention, we should fix the server code. If not, we need to document it in 
NEWS.txt so that operators are not surprised.

> Remove joda time from dependencies
> ----------------------------------
>
>                 Key: CASSANDRA-15257
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15257
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Dependencies
>            Reporter: Boris Tsirkin
>            Assignee: Boris Tsirkin
>            Priority: Low
>              Labels: pull-request-available
>             Fix For: 4.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Joda time is not supported anymore
> {quote}
> Note that Joda-Time is considered to be a largely “finished” project. No 
> major enhancements are planned. If using Java SE 8, please migrate to 
> {{java.time}} (JSR-310).
> {quote}
> See: https://www.joda.org/joda-time/



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to