Skip to content

Added large data sets tests.#78

Merged
pcmanus merged 1 commit into
apache:masterfrom
joaquincasares:large_data_sets_pr
May 7, 2013
Merged

Added large data sets tests.#78
pcmanus merged 1 commit into
apache:masterfrom
joaquincasares:large_data_sets_pr

Conversation

@joaquincasares
Copy link
Copy Markdown

Added the following tests:

  • testWideRows
  • testWideBatchRows
  • testByteRows
  • testLargeText
  • testWideTable

As well as mixedDurationTest which randomly chooses 10 tests from the above set to run.

@pcmanus
Copy link
Copy Markdown

pcmanus commented May 6, 2013

Not convinced about that mixedDurationTest. I'm absolutely good having a "short" and "long" target, because it's useful during development to be able to run some sanity check before committing, but the long test needs to be run by jenkins or the like sooner or later, so I'm not sure what running a random set of tests really solve.

@joaquincasares
Copy link
Copy Markdown
Author

We want to implement "duration tests" before the next release and the current test will be a foundation for these types of tests.

Ideally, the duration tests will run for at least 72 hours and ensure that the driver continues correctly, without fail. I'm -1 about having these 72 hour tests grouped up with the "long tests" since there is no need for Jenkins to run these on each build.

Ideally, the short tests are for sanity checks, the long tests are for Jenkins, and the duration tests will be run before each major release.

@joaquincasares joaquincasares mentioned this pull request May 6, 2013
pcmanus pushed a commit that referenced this pull request May 7, 2013
@pcmanus pcmanus merged commit 0f5b5bd into apache:master May 7, 2013
@pcmanus
Copy link
Copy Markdown

pcmanus commented May 7, 2013

For the record, I'm still not convinced by mixedDurationTest.

@joaquincasares joaquincasares deleted the large_data_sets_pr branch May 7, 2013 20:38
@joaquincasares
Copy link
Copy Markdown
Author

+1 on not being entirely convinced. I would ideally increase the number within the for loop to run longer multiple day tests and get rid of the cluster discarding prior to an official test.

For now that runs for a "small duration", if you will, but we can hit the ground running to adapt the test when needed.

Sfurti-yb pushed a commit to yugabyte/cassandra-java-driver that referenced this pull request Dec 8, 2023
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