Ah crap. Yes it should be. Was rushing to get fixes in for 3M. Can we just
worry about it at the next code change (and there will be more coming up
in the near future, believe me :-)).

Cheers,
Mark.


On 30/04/14 1:15 am, "Matt Tucker" <notifications@github.com> wrote:

>In src/org/pentaho/cassandra/legacy/CassandraColumnMetaData.java:
>> @@ -958,6 +1006,11 @@ public static String
>>getCassandraTypeForValueMeta( ValueMetaInterface vm ) {
>>        case ValueMetaInterface.TYPE_NUMBER:
>>          return "DoubleType"; //$NON-NLS-1$
>>        case ValueMetaInterface.TYPE_DATE:
>> +
>> +        // TimestampType is only present in Cassandra 2.x
>> +        if ( getTimestampType() != null ) {
>> +          return "TimestampType";
>Should this be an NLS string as well?
>—
>Reply to this email directly or view it on GitHub
><https://github.com/pentaho/pentaho-cassandra-plugin/pull/30/files#r120917
>05>.