Cassandra
  1. Cassandra
  2. CASSANDRA-5918

Remove CQL2 entirely from Cassandra 3.0

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Fix Version/s: 3.0
    • Component/s: None

      Description

      CQL2 is officially no longer worked on since 1.2. cqlsh no longer supports CQL2 as of Cassandra 2.0.

      It's probably the time to deprecate CQL2 in 2.0 and to remove it entirely in 2.2 - there is nothing in CQL2 now that can't be done via CQL3 and two versions advance warning is plenty of time for those few still using CQL2 to switch to CQL3.

        Activity

        Aleksey Yeschenko created issue -
        Hide
        Aleksey Yeschenko added a comment -

        Added the deprecation warning to 2.0 in 5fe46e145adfe54a1fc4521fd274833e3bce4ac2.

        Show
        Aleksey Yeschenko added a comment - Added the deprecation warning to 2.0 in 5fe46e145adfe54a1fc4521fd274833e3bce4ac2.
        Hide
        Patrick McFadin added a comment -

        If we are going to remove it as a breaking change, this should be 3.0. There are still people in production using CQL2. If 2.x issues a warning, then we can give them until 3.x to migrate away.

        Show
        Patrick McFadin added a comment - If we are going to remove it as a breaking change, this should be 3.0. There are still people in production using CQL2. If 2.x issues a warning, then we can give them until 3.x to migrate away.
        Hide
        Brandon Williams added a comment -

        That's basically what 2.2 is, "not the next major release, but the one after that."

        Show
        Brandon Williams added a comment - That's basically what 2.2 is, "not the next major release, but the one after that."
        Hide
        Jonathan Ellis added a comment -

        If it takes calling it 3.0 to make you happy, that's doable.

        Show
        Jonathan Ellis added a comment - If it takes calling it 3.0 to make you happy, that's doable.
        Hide
        Patrick McFadin added a comment -

        Your concern for my happiness is heart warming. heh

        Whatever it is that isn't "we just took away functionality inside 2.x" will be just fine.

        Show
        Patrick McFadin added a comment - Your concern for my happiness is heart warming. heh Whatever it is that isn't "we just took away functionality inside 2.x" will be just fine.
        Hide
        Robert Coli added a comment -

        FWIW, this sounds like a reasonable idea to me. It is also encouraging as a model for deprecating features which some small set of people might be using now but which no one should start using because they're unsupported/unmaintained.

        Show
        Robert Coli added a comment - FWIW, this sounds like a reasonable idea to me. It is also encouraging as a model for deprecating features which some small set of people might be using now but which no one should start using because they're unsupported/unmaintained.
        Aleksey Yeschenko made changes -
        Field Original Value New Value
        Summary Remove CQL2 entirely from Cassandra 2.2 Remove CQL2 entirely from Cassandra 3.0
        Hide
        Aleksey Yeschenko added a comment -

        Pushed the commit to https://github.com/iamaleksey/cassandra/commits/5918.

        Not sure what to do with thirft definitions (besides obviously bumping the major). Throw IRE for all the CQL2 methods (as in the commit), or get rid of them altogether, erasing the last and only traces of CQL2 existence (other than NEWS and CHANGES).

        Show
        Aleksey Yeschenko added a comment - Pushed the commit to https://github.com/iamaleksey/cassandra/commits/5918 . Not sure what to do with thirft definitions (besides obviously bumping the major). Throw IRE for all the CQL2 methods (as in the commit), or get rid of them altogether, erasing the last and only traces of CQL2 existence (other than NEWS and CHANGES).
        Aleksey Yeschenko made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Reviewer Sylvain Lebresne [ slebresne ]
        Aleksey Yeschenko made changes -
        Reviewer Sylvain Lebresne [ slebresne ] Jonathan Ellis [ jbellis ]
        Hide
        Jonathan Ellis added a comment -

        I'd say it's marginally more user-friendly to throw IRE than to rely on the Thrift server to respond with whatever its no-such-method exception is.

        Show
        Jonathan Ellis added a comment - I'd say it's marginally more user-friendly to throw IRE than to rely on the Thrift server to respond with whatever its no-such-method exception is.
        Hide
        Aleksey Yeschenko added a comment -

        Left them alone (throwing exceptions instead of removing entirely). That, and regenerated thrift (tiny change). Pushed -f to the same branch.

        Show
        Aleksey Yeschenko added a comment - Left them alone (throwing exceptions instead of removing entirely). That, and regenerated thrift (tiny change). Pushed -f to the same branch.
        Hide
        Jonathan Ellis added a comment -

        Ship it!

        Show
        Jonathan Ellis added a comment - Ship it!
        Hide
        Aleksey Yeschenko added a comment -

        Committed, thanks.

        Show
        Aleksey Yeschenko added a comment - Committed, thanks.
        Aleksey Yeschenko made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Aleksey Yeschenko made changes -
        Labels cql cql docs-impacting
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        180d 2h 49m 1 Aleksey Yeschenko 18/Feb/14 03:59
        Patch Available Patch Available Resolved Resolved
        2d 15h 39m 1 Aleksey Yeschenko 20/Feb/14 19:39

          People

          • Assignee:
            Aleksey Yeschenko
            Reporter:
            Aleksey Yeschenko
            Reviewer:
            Jonathan Ellis
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development