Skip to content

Commit e8452ca

Browse files
committed
JDBC store testing with new jcstress suite
1 parent 240d207 commit e8452ca

File tree

7 files changed

+230
-362
lines changed

7 files changed

+230
-362
lines changed

quickfixj-core/src/main/java/quickfix/JdbcStore.java

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -194,32 +194,6 @@ public int getNextTargetMsgSeqNum() throws IOException {
194194
return cache.getNextTargetMsgSeqNum();
195195
}
196196

197-
int getNextSenderMsgSeqNumFromDb() throws SQLException {
198-
return getNextMsgSeqNumsFromDb()[0];
199-
}
200-
201-
int getNextTargetMsgSeqNumFromDb() throws SQLException {
202-
return getNextMsgSeqNumsFromDb()[1];
203-
}
204-
205-
int[] getNextMsgSeqNumsFromDb() throws SQLException {
206-
try (Connection connection = dataSource.getConnection()) {
207-
try (PreparedStatement query = connection.prepareStatement(SQL_GET_SEQNUMS)) {
208-
setSessionIdParameters(query, 1);
209-
210-
try (ResultSet result = query.executeQuery()) {
211-
if (result.next()) {
212-
int targetSeqNum = result.getInt(2);
213-
int senderSeqNum = result.getInt(3);
214-
return new int[] {senderSeqNum, targetSeqNum};
215-
} else {
216-
return new int[] {-1, -1};
217-
}
218-
}
219-
}
220-
}
221-
}
222-
223197
public void incrNextSenderMsgSeqNum() throws IOException {
224198
cache.incrNextSenderMsgSeqNum();
225199
setNextSenderMsgSeqNum(cache.getNextSenderMsgSeqNum());

quickfixj-stress-test/README.md

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -60,15 +60,13 @@ $ java -jar quickfixj-stress-test/target/quickfixj-stress-test.jar -h
6060

6161
Some useful jcstress options include:
6262

63+
- `h` - Display help
6364
- `-t <test>` - Run specific test or test group
6465
- `-v` - Verbose mode
65-
- `-time <ms>` - Duration for each test iteration
66-
- `-iters <n>` - Number of iterations
6766
- `-jvmArgs <args>` - Additional JVM arguments
6867

6968
## Additional Notes
7069

71-
- Test results are generated in the `results/` directory by default
7270
- For CI/CD integration, use the full path to the JAR file
7371
- Stress tests may take significant time to complete depending on your system resources
7472
- Review jcstress documentation for advanced configuration options
@@ -77,7 +75,6 @@ Some useful jcstress options include:
7775

7876
If you encounter issues:
7977

80-
1. Ensure all prerequisites are met
81-
2. Verify the JAR file was built successfully in `quickfixj-stress-test/target/`
82-
3. Check that you have sufficient system resources (CPU, memory)
83-
4. Review the jcstress output for specific error messages
78+
- Verify the JAR file was built successfully in `quickfixj-stress-test/target/`
79+
- Check that you have sufficient system resources (CPU, memory)
80+
- Review the jcstress output for specific error messages

quickfixj-stress-test/pom.xml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,6 @@
2828
<artifactId>jcstress-core</artifactId>
2929
<version>${jcstress.version}</version>
3030
</dependency>
31-
<dependency>
32-
<groupId>hsqldb</groupId>
33-
<artifactId>hsqldb</artifactId>
34-
<version>${hsqldb.version}</version>
35-
</dependency>
3631
<dependency>
3732
<groupId>org.slf4j</groupId>
3833
<artifactId>slf4j-jdk14</artifactId>

quickfixj-stress-test/src/main/java/quickfix/JdbcStoreHsqldbStressTest.java

Lines changed: 0 additions & 214 deletions
This file was deleted.

0 commit comments

Comments
 (0)