Log workAgile BoardRank to TopRank to BottomAttach filesAttach ScreenshotVotersWatch issueWatchersConvert to IssueMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

      Description

      This is followup to CASSANDRA-10289

      The tests currently failing should be:

      • cqlshlib.test.test_cqlsh_completion.TestCqlshCompletion.test_complete_in_create_columnfamily
        • uses create_columnfamily_table_template. Stefania says "the ( after CREATE ... IF does not look valid to me."
      • cqlshlib.test.test_cqlsh_completion.TestCqlshCompletion.test_complete_in_create_table
        • uses create_columnfamily_table_template, see above.
      • cqlshlib.test.test_cqlsh_completion.TestCqlshCompletion.test_complete_in_delete
        • Stefania says: "I don't think keyspaces are a valid completion after DELETE a [ and after DELETE FROM twenty_rows_composite_table USING TIMESTAMP 0 WHERE TOKEN(a) >=. From a quick analysis of cqlhandling.py I think it comes from <term>, which picks up <functionName>, which was changed to include ks. by CASSANDRA-7556.
      • cqlshlib.test.test_cqlsh_completion.TestCqlshCompletion.test_complete_in_drop_keyspace
        • Stefania says: "the ; after DROP KEYSPACE IF is not valid.
      • cqlshlib.test.test_cqlsh_output.TestCqlshOutput.test_timestamp_output

      I'm happy to break these out into separate tickets if necessary.

      To run the tests locally, I cd to cassandra/pylib/cqlshlib and run the following:

      ccm create -n 1 --install-dir=../.. test
      ccm start --wait-for-binary-proto
      nosetests test 2>&1
      ccm remove
      

      This requires nose and ccm. Until CASSANDRA-10289 is resolved, you'll have to use my branch here: https://github.com/mambocab/cassandra/tree/fix-cqlsh-tests

      Tests for this branch are run (non-continuously) here:

      http://cassci.datastax.com/job/scratch_mambocab-fix_cqlsh/

      Assigning Stefania Alborghetti for now, since she's already looked at 10289, but feel free to reassign.

        Attachments

          Activity

          $i18n.getText('security.level.explanation', $currentSelection) Viewable by All Users
          Cancel

            People

            • Assignee:
              stefania Stefania Alborghetti Assign to me
              Reporter:
              mambocab Jim Witschey
              Authors:
              Stefania Alborghetti
              Reviewers:
              Benjamin Lerer
              Tester:
              Jim Witschey

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment