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

Show Schema; inserts an extra comma in column_metadata

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Low
    • Resolution: Fixed
    • 1.0.7
    • None
    • Low

    Description

      create column family inode
      with column_type = 'Standard'
      and comparator = 'DynamicCompositeType(t=>org.apache.cassandra.db.marshal.TimeUUIDType,s=>org.apache.cassandra.db.marshal.UTF8Type,b=>org.apache.cassandra.db.marshal.BytesType)'
      and default_validation_class = 'BytesType'
      and key_validation_class = 'BytesType'
      and rows_cached = 0.0
      and row_cache_save_period = 0
      and row_cache_keys_to_save = 2147483647
      and keys_cached = 1000000.0
      and key_cache_save_period = 14400
      and read_repair_chance = 1.0
      and gc_grace = 60
      and min_compaction_threshold = 4
      and max_compaction_threshold = 32
      and replicate_on_write = true
      and row_cache_provider = 'ConcurrentLinkedHashCacheProvider'
      and compaction_strategy = 'org.apache.cassandra.db.compaction.SizeTieredCompactionStrategy'
      and comment = 'Stores file meta data'
      and column_metadata = [
      {column_name : 'b@70617468',
      validation_class : BytesType,
      index_name : 'path',
      index_type : 0,
      },
      {column_name : 'b@73656e74696e656c',
      validation_class : BytesType,
      index_name : 'sentinel',
      index_type : 0,
      },
      {column_name : 'b@706172656e745f70617468',
      validation_class : BytesType,
      index_name : 'parent_path',
      index_type : 0,
      }];

      That's that was outputted when I ran show schema. When I tried it on a new cluster, it failed since the commas after 'index_type: 0' were present.

      Proposed fixes:
      1. Allow trailing commas
      2. Do not output trailing commas

      Attachments

        1. CASSANDRA-1.0-3714.txt
          2 kB
          Yuki Morishita

        Activity

          People

            yukim Yuki Morishita
            j.casares Joaquin Casares
            Yuki Morishita
            Pavel Yaskevich
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: