[ https://issues.apache.org/jira/browse/CASSANDRA-4163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13256957#comment-13256957 ]
Jonathan Ellis commented on CASSANDRA-4163: ------------------------------------------- LGTM. Nit: could we initialize properties to an empty map to avoid having to null-check it? > CQL3 ALTER TABLE command causes NPE > ----------------------------------- > > Key: CASSANDRA-4163 > URL: https://issues.apache.org/jira/browse/CASSANDRA-4163 > Project: Cassandra > Issue Type: Bug > Components: Core > Affects Versions: 1.1.0 > Environment: INFO 16:07:11,757 Cassandra version: 1.1.0-rc1-SNAPSHOT > INFO 16:07:11,757 Thrift API version: 19.30.0 > INFO 16:07:11,758 CQL supported versions: 2.0.0,3.0.0-beta1 (default: 2.0.0) > Reporter: Kristine Hahn > Assignee: paul cannon > Labels: cql3 > Fix For: 1.1.0 > > Attachments: 4163.patch.txt > > > To reproduce the problem: > ./cqlsh --cql3 > Connected to Test Cluster at localhost:9160. > [cqlsh 2.2.0 | Cassandra 1.1.0-rc1-SNAPSHOT | CQL spec 3.0.0 | Thrift > protocol 19.30.0] > Use HELP for help. > cqlsh> CREATE KEYSPACE test34 WITH strategy_class = > 'org.apache.cassandra.locator.SimpleStrategy' AND > strategy_options:replication_factor='1'; > cqlsh> USE test34; > cqlsh:test34> CREATE TABLE users ( > ... password varchar, > ... gender varchar, > ... session_token varchar, > ... state varchar, > ... birth_year bigint, > ... pk varchar, > ... PRIMARY KEY (pk) > ... ); > cqlsh:test34> ALTER TABLE users ADD coupon_code varchar; > TSocket read 0 bytes -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira