Cassandra
  1. Cassandra
  2. CASSANDRA-2743

add TABLE as a CQL alias for COLUMNFAMILY

    Details

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

      Description

      This will make it easier on tools that support multiple databases like SQLAlchemy.

      1. CASSANDRA-2743.patch
        3 kB
        Pavel Yaskevich

        Activity

        Hide
        Rick Shaw added a comment -

        And SCHEMA for KeySpace"?

        Show
        Rick Shaw added a comment - And SCHEMA for KeySpace"?
        Hide
        Jonathan Ellis added a comment -

        Sure.

        Show
        Jonathan Ellis added a comment - Sure.
        Hide
        Pavel Yaskevich added a comment -

        added both SCHEMA and TABLE aliases + modified tests to check if SCHEMA/TABLE is legal.

        Show
        Pavel Yaskevich added a comment - added both SCHEMA and TABLE aliases + modified tests to check if SCHEMA/TABLE is legal.
        Hide
        Jonathan Ellis added a comment -

        I'm going to leave this open a couple days for comment – I know Eric was not fond of the idea over in the original CQL syntax ticket, so I don't want to rush this through while he's traveling.

        Show
        Jonathan Ellis added a comment - I'm going to leave this open a couple days for comment – I know Eric was not fond of the idea over in the original CQL syntax ticket, so I don't want to rush this through while he's traveling.
        Hide
        Eric Evans added a comment -

        I wasn't (still aren't) fond of the idea of CQL driving some general shift in terminology, but I was under the impression that this ticket was about creating an alias to be friendlier to existing tools.

        So long as we continue to use consistent terminology in public discussions, documentation, etc, I don't see a problem with having an alias.

        If I've misunderstood "alias" here, and it is about changing the terminology, then we should probably move this to the mailing list. If that happens you should probably be prepared to clear your schedule for the next few days.

        Show
        Eric Evans added a comment - I wasn't (still aren't) fond of the idea of CQL driving some general shift in terminology, but I was under the impression that this ticket was about creating an alias to be friendlier to existing tools. So long as we continue to use consistent terminology in public discussions, documentation, etc, I don't see a problem with having an alias. If I've misunderstood "alias" here, and it is about changing the terminology, then we should probably move this to the mailing list. If that happens you should probably be prepared to clear your schedule for the next few days.
        Hide
        Jonathan Ellis added a comment - - edited

        sounds good, committed

        Show
        Jonathan Ellis added a comment - - edited sounds good, committed
        Hide
        Hudson added a comment -

        Integrated in Cassandra-0.8 #162 (See https://builds.apache.org/job/Cassandra-0.8/162/)
        add SCHEMA/TABLE as aliases for KS/CF
        patch by pyaskevich; reviewed by jbellis for CASSANDRA-2743

        jbellis : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1134110
        Files :

        • /cassandra/branches/cassandra-0.8/test/system/test_cql.py
        • /cassandra/branches/cassandra-0.8/CHANGES.txt
        • /cassandra/branches/cassandra-0.8/src/java/org/apache/cassandra/cql/Cql.g
        Show
        Hudson added a comment - Integrated in Cassandra-0.8 #162 (See https://builds.apache.org/job/Cassandra-0.8/162/ ) add SCHEMA/TABLE as aliases for KS/CF patch by pyaskevich; reviewed by jbellis for CASSANDRA-2743 jbellis : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1134110 Files : /cassandra/branches/cassandra-0.8/test/system/test_cql.py /cassandra/branches/cassandra-0.8/CHANGES.txt /cassandra/branches/cassandra-0.8/src/java/org/apache/cassandra/cql/Cql.g

          People

          • Assignee:
            Pavel Yaskevich
            Reporter:
            Jonathan Ellis
            Reviewer:
            Jonathan Ellis
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development