-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Automated tests - Circle CI #2
Comments
CircleCI is the way to go as discussed. Do you want me to pick this up? @rnorth |
Yes please, but can we talk about it later today (my time) / tomorrow (your On Fri, Jun 26, 2015 at 9:18 AM, gusohal [email protected] wrote:
|
Sure |
I started playing around with this on a branch (circle-ci) but it's a little more complicated than first meets the eye - because we need to connect to the docker daemon via tcp, and that entails getting TLS set up. This article (in German) seems to address exactly this scenario, so will probably be quite useful: |
Oooh fun! |
Now closed in df0f41b |
Many enterprise environments require a JDBC connection to Cassandra for integration into legacy systems. This patch changes the parent of the CassandraContainer to JdbcDatabaseContainer and optionally allows enables the 'native api' (thrift-over-rcp) used by many JDBC driver implementations. A new test was added - testCassandraJdbcQuery(). == Status This code isn't ready for merge - it's being offered for review. It loosely corresponds to my work on Friday creating a local fork that integrates into our existing environment where we were already using JDBC connections to Cassandra servers. That fork works so I know this fork is very close to working. Of particular note is the log line ``` INFO [main] 2019-12-14 21:31:33,520 ThriftServer.java:116 - Binding thrift service to /0.0.0.0:9160 ``` since that indicates that the server is listening to the RPC port required by the JDBC driver. == Remaining Work testcontainers#1 The connection attempt fails with ``` Caused by: com.datastax.driver.core.exceptions.NoHostAvailableException: All host(s) tried for query failed (tried: localhost/127.0.0.1:32862 (com.datastax.driver.core.exceptions.TransportException: [localhost/127.0.0.1:32862] Channel has been closed)) at com.datastax.driver.core.ControlConnection.reconnectInternal(ControlConnection.java:268) at com.datastax.driver.core.ControlConnection.connect(ControlConnection.java:107) at com.datastax.driver.core.Cluster$Manager.negotiateProtocolVersionAndConnect(Cluster.java:1813) at com.datastax.driver.core.Cluster$Manager.init(Cluster.java:1726) at com.datastax.driver.core.Cluster.init(Cluster.java:214) at com.datastax.driver.core.Cluster.connectAsync(Cluster.java:387) at com.datastax.driver.core.Cluster.connect(Cluster.java:338) at com.github.adejanovski.cassandra.jdbc.SessionHolder.createSession(SessionHolder.java:201) ... 36 more ``` however I know that the same JDBC driver works elsewhere. It may be something as simple as a missing datastax jar. == Remaining Work testcontainers#2 The test includes calls to `cassandraContainer.setWaitStrategy()`. Neither seems to have any effect. In the other fork I only saw a wait strategy used was when I explicitly set the waitStrategy value in the constructor.
If possible, use Travis, although use of docker will probably make this complex...
The text was updated successfully, but these errors were encountered: