[
https://issues.apache.org/jira/browse/DRILL-7982?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17394157#comment-17394157
]
ASF GitHub Bot commented on DRILL-7982:
---------------------------------------
vdiravka commented on a change in pull request #2287:
URL: https://github.com/apache/drill/pull/2287#discussion_r683601103
##########
File path:
contrib/storage-cassandra/src/test/java/org/apache/drill/exec/store/cassandra/TestCassandraSuit.java
##########
@@ -65,7 +66,12 @@ public static void tearDownCluster() {
private static void startCassandra() {
cassandra = new CassandraContainer<>("cassandra")
- .withInitScript("queries.cql");
+ .withInitScript("queries.cql")
+ .withStartupTimeout(Duration.ofMinutes(3))
+ .withEnv("CASSANDRA_SNITCH",
"GossipingPropertyFileSnitch") // Tune Cassandra options for faster startup
+ .withEnv("JVM_OPTS",
"-Dcassandra.skip_wait_for_gossip_to_settle=0 -Dcassandra.initial_token=0")
+ .withEnv("HEAP_NEWSIZE", "128M")
+ .withEnv("MAX_HEAP_SIZE", "1024M");
Review comment:
> The Cassandra tests seem to keep timing out and breaking the CI. My
recollection of the Cassandra plugin was that Cassandra really does take a long
time to start up, possibly more than 1 minute. This was why we implemented it
to preserve the connection as long as possible rather than creating tons of new
connections.
I am aware about timing out test of container for Cassandra plugin. I also
fixed that several days ago in my branch. But I didn't merge it, because this
is not only the one issue on CI and don't fix CI build. For instance the CI
build is restarted several times for this PR and still not green.
--------------------------------------------------------------------------------------------------------------------------------------------
> To my knowledge, the Splunk ones are working fine in the CI.
My question was about memory options only in the above comment and
`luocooong` answered that he is going to eliminate them.
--------------------------------------------------------------------------------------------------------------------------------------------
> It makes me wonder, should we add the GossipingPropertyFileSnitch
parameter to the Cassandra plugin?
[GossipingPropertyFileSnitch](https://github.com/PaytmLabs/cassandra/blob/master/src/java/org/apache/cassandra/locator/GossipingPropertyFileSnitch.java)
looks Cassandra specific code only.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
> Timed out waiting for container port to open
> --------------------------------------------
>
> Key: DRILL-7982
> URL: https://issues.apache.org/jira/browse/DRILL-7982
> Project: Apache Drill
> Issue Type: Task
> Components: Tools, Build & Test
> Reporter: Cong Luo
> Assignee: Cong Luo
> Priority: Major
> Fix For: 1.20.0
>
>
> If the default 60s timeout is not sufficient, it can be altered with the
> {{withStartupTimeout()}} method.
> See also [Waiting for containers to start or be
> ready|https://www.testcontainers.org/features/startup_and_waits/]
> The java stack trace logs :
> {code:java}
> Error: org.apache.drill.exec.store.cassandra.TestCassandraSuit Time
> elapsed: 190.343 s <<< ERROR!
> 16634org.testcontainers.containers.ContainerLaunchException: Container
> startup failed
> 16635 at
> org.apache.drill.exec.store.cassandra.TestCassandraSuit.startCassandra(TestCassandraSuit.java:69)
> 16636 at
> org.apache.drill.exec.store.cassandra.TestCassandraSuit.initCassandra(TestCassandraSuit.java:46)
> 16637Caused by: org.rnorth.ducttape.RetryCountExceededException: Retry limit
> hit with exception
> 16638 at
> org.apache.drill.exec.store.cassandra.TestCassandraSuit.startCassandra(TestCassandraSuit.java:69)
> 16639 at
> org.apache.drill.exec.store.cassandra.TestCassandraSuit.initCassandra(TestCassandraSuit.java:46)
> 16640Caused by: org.testcontainers.containers.ContainerLaunchException: Could
> not create/start container
> 16641 at
> org.apache.drill.exec.store.cassandra.TestCassandraSuit.startCassandra(TestCassandraSuit.java:69)
> 16642 at
> org.apache.drill.exec.store.cassandra.TestCassandraSuit.initCassandra(TestCassandraSuit.java:46)
> 16643Caused by: org.testcontainers.containers.ContainerLaunchException: Timed
> out waiting for container port to open (localhost ports: [49174] should be
> listening)
> 16644 at
> org.apache.drill.exec.store.cassandra.TestCassandraSuit.startCassandra(TestCassandraSuit.java:69)
> 16645 at
> org.apache.drill.exec.store.cassandra.TestCassandraSuit.initCassandra(TestCassandraSuit.java:46)
> {code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
