Possible inconsistent CQL Identifier to string in ListTables and Metadata commands

[amorton]

The following was added to the CqlIdentifierUtil class

  /** Remove the quotes from the identifier */
  public static String externalRepresentation(CqlIdentifier identifier) {
    return identifier.asInternal();
  }

But we already have

  public static String cqlIdentifierToStringForUser(CqlIdentifier identifier) {
    return identifier.asCql(true);
  }

Why do we want to remove the double quotes ? And if we added this function did we consider removing the other one ?

[amorton]

If we do need both, i.e. one way for messages and one for JSON fields names, then we should improve the method names to make it clearer

[amorton]

Suggestion from chat with @tatu-at-datastax for name such as

toCql
toJSON
toMessage

fromJSON
fromCQL