frankgh commented on code in PR #14:
URL:
https://github.com/apache/cassandra-analytics/pull/14#discussion_r1337851038
##########
cassandra-analytics-core-example/README.md:
##########
@@ -108,8 +110,8 @@ cassandra_instances:
I have a 3 node setup, so I configure Sidecar for those 3 nodes. CCM creates
the Cassandra cluster under
`${HOME}/.ccm/test`, so I update my `data_dirs` and `staging_dir`
configuration to use my local path.
-Finally, run Cassandra Sidecar, we skip running integration tests because we
need docker for integration tests. You
-can opt to run integration tests if you have docker running in your local
environment.
+Finally, run Cassandra Sidecar, we skip running integration tests because the
integration tests take quite some time.
+You can, of course, chose to run them (and should when working on the sidecar
the project itself)
Review Comment:
```suggestion
You can, of course, choose to run them (and should when working on the
sidecar the project itself)
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org
