Skip to content

Conversation

@ca61688
Copy link
Collaborator

@ca61688 ca61688 commented Oct 2, 2025

Make sure you have checked all steps below.

Issue

  • My PR fully resolves the following issues. I've referenced an issue in the PR title, for example "Issue 1234 - My
    Feature". Note that before an issue is finished, you can still make a pull request by raising a separate issue
    for your progress.

Tests

  • My PR adds the following tests based on our test strategy OR does not need testing for this extremely good reason:
    • MyUnitTest

Documentation

  • In case of new functionality, my PR adds documentation that describes how to use it, or I have linked to a
    separate issue for that below.
  • If I have added new Java code, I have added Javadoc that explains it following our conventions and style.
  • If I have added or removed any dependencies from the project, I have updated the NOTICES file.

@ca61688 ca61688 linked an issue Oct 2, 2025 that may be closed by this pull request

@SystemTestSuite
@ExcludeTags({"slow", "expensive"})
@ExcludeTags({"slow1", "slow2", "slow3", "expensive1", "expensive2", "expensive3"})
Copy link
Collaborator

Choose a reason for hiding this comment

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

We could avoid stating all the numbers here if we give the slow and expensive annotations a "slow" and "expensive" tag as well as the numbered tags.


@SystemTestSuite
@ExcludeTags("expensive")
@ExcludeTags({"expensive1", "expensive2", "expensive3"})
Copy link
Collaborator

Choose a reason for hiding this comment

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

We could avoid stating all the numbers here if we give the expensive annotations an "expensive" tag as well as the numbered tags.

wait
elif [ "$MAIN_SUITE_NAME" == "functional" ]; then
echo "Running slow tests in parallel. Start time: [$(time_str)]"
runSlowTests $@
Copy link
Collaborator

Choose a reason for hiding this comment

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

This $@ needs quoting as in the previous comment #5762 (comment).

sudo cp -r scripts quick/scripts
sudo cp -r python quick/python
sudo cp -r code-style quick/code-style
sudo cp -r rust quick/rust
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there a reason to copy the individual folders explicitly like this, rather than copy everything? I could see excluding the Git folder so we don't need to copy the whole Git repository. It might be easier to work with if we either just copy everything, or specifically exclude things rather than include them?

Following up on my previous comment #5762 (comment).

pushd $SLEEPER_DIR
#Make copies of the java folder to run independent maven builds in parallel
sudo rm -rf quick
mkdir quick
Copy link
Collaborator

Choose a reason for hiding this comment

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

Might it be worth putting the copies under a separate folder to make sure there can't be any naming conflicts between the test suites and the real folders?

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.

Parallelise nightly tests across multiple processes

3 participants