Uploaded image for project: 'Cassandra'
  1. Cassandra
  2. CASSANDRA-2743

add TABLE as a CQL alias for COLUMNFAMILY

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Fix Version/s: 0.8.1
    • Component/s: CQL
    • 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
        ardot Rick Shaw added a comment -

        And SCHEMA for KeySpace"?

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

        Sure.

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

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

        Show
        xedin Pavel Yaskevich added a comment - added both SCHEMA and TABLE aliases + modified tests to check if SCHEMA/TABLE is legal.
        Hide
        jbellis 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
        jbellis 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
        urandom 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
        urandom 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
        jbellis Jonathan Ellis added a comment - - edited

        sounds good, committed

        Show
        jbellis Jonathan Ellis added a comment - - edited sounds good, committed
        Hide
        hudson 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 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:
            xedin Pavel Yaskevich
            Reporter:
            jbellis Jonathan Ellis
            Reviewer:
            Jonathan Ellis
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development