Skip to content

Conversation

@gagan405
Copy link

Skipping batchReadTTL test for older server versions: 7.0 or below

Copy link
Member

@BrianNichols BrianNichols left a comment

Choose a reason for hiding this comment

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

The retrieval of the server node version in isServerVersionGreaterThan7_0() should not be performed on every test. Instead, store the server version on test suite initialization and then compare that version in each test that needs it.

Copy link
Member

@BrianNichols BrianNichols left a comment

Choose a reason for hiding this comment

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

Sorry, I should have been more specific. serverVersion should be stored in the com.aerospike.test.util.Args class in setServerSpecific(). setServerSpecific() already retrieves the server version for other purposes. The Args class is accessible by both sync and async tests.

@gagan405 gagan405 force-pushed the CLIENT-3839-skip-test-for-compatibility-with-verion-7 branch from bcd5fc2 to d692bfd Compare October 27, 2025 20:14
@BrianNichols BrianNichols merged commit 82885e4 into aerospike:stage Oct 27, 2025
4 checks passed
@gagan405 gagan405 deleted the CLIENT-3839-skip-test-for-compatibility-with-verion-7 branch October 27, 2025 20:45
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.

2 participants