Skip to content
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

[dtest]: test_connectivity_with_cassandra_stress failed with java.lang.NoSuchMethodError #266

Conversation

yaronkaikov
Copy link

@yaronkaikov yaronkaikov commented Dec 21, 2023

Following the changes made in
scylladb/scylla-tools-java#369, dtest is now failing during test_connectivity_with_cassandra_stress with the following error:

stderr: java.lang.NoSuchMethodError: 'java.lang.String com.google.common.net.HostAndPort.getHostText()'
	at com.datastax.driver.core.ScyllaCloudDatacenter.getServer(ScyllaCloudDatacenter.java:77)
	at com.datastax.driver.core.Cluster$Builder.withScyllaCloudConnectionConfig(Cluster.java:1404)
	at com.datastax.driver.core.Cluster$Builder.withScyllaCloudConnectionConfig(Cluster.java:1397)
	at com.datastax.driver.core.Cluster$Builder.withScyllaCloudConnectionConfig(Cluster.java:1393)
	at com.datastax.driver.core.Cluster$Builder.withScyllaCloudConnectionConfig(Cluster.java:1389)
	at org.apache.cassandra.stress.util.JavaDriverClient.connect(JavaDriverClient.java:181)
	at org.apache.cassandra.stress.settings.StressSettings.getJavaDriverClient(StressSettings.java:213)
	at org.apache.cassandra.stress.settings.SettingsSchema.createKeySpacesNative(SettingsSchema.java:84)
	at org.apache.cassandra.stress.settings.SettingsSchema.createKeySpaces(SettingsSchema.java:74)
	at org.apache.cassandra.stress.settings.StressSettings.maybeCreateKeyspaces(StressSettings.java:230)
	at org.apache.cassandra.stress.StressAction.run(StressAction.java:58)
	at org.apache.cassandra.stress.Stress.run(Stress.java:143)
	at org.apache.cassandra.stress.Stress.main(Stress.java:62)

@yaronkaikov yaronkaikov requested a review from muzarski December 21, 2023 08:11
@yaronkaikov yaronkaikov force-pushed the update-guava-deprecated-methods branch 5 times, most recently from c2f4b9e to 2bfdafd Compare December 21, 2023 08:53
@yaronkaikov yaronkaikov changed the title Rename getHostText() to getHost() [dtest]: test_connectivity_with_cassandra_stress failed with java.lang.NoSuchMethodError Dec 21, 2023
@yaronkaikov yaronkaikov force-pushed the update-guava-deprecated-methods branch from 2bfdafd to 8d8070e Compare December 21, 2023 08:58
Copy link

@muzarski muzarski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please adjust the commit message so it describes introduced change. Otherwise LGTM.

@muzarski
Copy link

@Lorak-mmk could you please take a look at it?

@mykaul
Copy link

mykaul commented Dec 21, 2023

Fix

return InetSocketAddress.createUnresolved(parsedServer.getHostText(), parsedServer.getPort());

In the same way as

(And update Guava, if we haven't done so in the Java driver, and update the Java driver in scylla-tools-java - cassandra-stress, if we haven't done so)

@yaronkaikov
Copy link
Author

yaronkaikov commented Dec 21, 2023

Fix

return InetSocketAddress.createUnresolved(parsedServer.getHostText(), parsedServer.getPort());

In the same way as

(And update Guava, if we haven't done so in the Java driver, and update the Java driver in scylla-tools-java - cassandra-stress, if we haven't done so)

Updated Guava , i assume we need to update the java driver after this will be merged

@yaronkaikov yaronkaikov force-pushed the update-guava-deprecated-methods branch 2 times, most recently from 64dfb37 to f0d3e38 Compare December 21, 2023 09:50
@yaronkaikov yaronkaikov force-pushed the update-guava-deprecated-methods branch from f0d3e38 to 51ff0d5 Compare December 21, 2023 09:51
@yaronkaikov
Copy link
Author

Closed by mistake
And can't re-open so i will open a new one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants