Counter table fix & docker build info for local docker test.

Author: pravinbhat

SIT/environment.sh

@@ -228,8 +228,17 @@ _Setup() {
 
   if [ "$(_testDockerCDM)" != "yes" ]; then  
     dockerContainerVersion=datastax/cassandra-data-migrator:${CDM_VERSION}
+
+    # Uncomment the below 'docker build' lines when making docker changes to ensure you test the docker changes
+    # Also comment the 'docker pull' line when 'docker build' is uncommented.
+    # Note this ('docker build') should be done only when testing docker changes locally (i.e. Do not commit)
+    # If you commit the 'docker build' step, the build will work but it will take too long as each time it will build
+    # CDM docker image instead of just downloading from DockerHub.
     _info "Pulling latest Docker container for ${dockerContainerVersion}"
     docker pull ${dockerContainerVersion}
+    # _info "Building latest Docker container for ${dockerContainerVersion}"
+    # docker build -t ${dockerContainerVersion} ..
+
     _info "Starting Docker container ${DOCKER_CASS}"
     docker run --name ${DOCKER_CDM} --network ${NETWORK_NAME} --ip ${SUBNET}.3 -e "CASS_USERNAME=${CASS_USERNAME}" -e "CASS_PASSWORD=${CASS_PASSWORD}" -e "CASS_CLUSTER=${DOCKER_CASS}" -d ${dockerContainerVersion}
     attempt=1

src/main/java/com/datastax/cdm/feature/WritetimeTTL.java

@@ -117,7 +117,6 @@ public boolean initializeAndValidate(CqlTable originTable, CqlTable targetTable)
                 logger.error("Counter table cannot specify TTL or WriteTimestamp columns as they cannot set on write");
                 isValid = false;
                 isEnabled = false;
-                return false;
             }
 
             logger.info("Counter table does not support TTL or WriteTimestamp columns as they cannot set on write, so feature is disabled");