[TEST] Rbroughan/jschmidt/s3v2/rc 9 but with parallelism at 1

[tryangul]

This PR + Testing to see if setting parallelism to 1 does anything

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
airbyte-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 11, 2025 0:12am

[edgao]

is this intentionally editing java-connector instead of bulk-connector.gradle? (that's the old connector plugin, which I'm pretty sure has no power here)

[edgao]

I'm fairly certain you want this instead

airbyte/build.gradle

Lines 183 to 189 in 09132c7

	if (concurrency.isInteger() && (concurrency as int) > 0) {
	// Define a fixed number of threads when the property is set to a positive integer.
	systemProperty 'junit.jupiter.execution.parallel.config.fixed.parallelism', concurrency
	} else {
	// Otherwise let JUnit manage the concurrency dynamically.
	systemProperty 'junit.jupiter.execution.parallel.config.strategy', 'dynamic'
	}

(or just edit the destination-s3/gradle.properties? I'm probably missing context, but that option is supposed to work)

[tryangul]

@edgao Sorry you're getting pinged for these, but these are just test permutations.

I will likely try the other permutations. If there's a good way to not notify you but get CI to run my tests, lmk.

If you dare to read further I'll explain myself.

I realize this is in the 'wrong spot'—but I'm kicking this off for reasons:

• last time we saw these tests get stuck, this is what I changed to get the tests unstuck
• I made this change in another pr to test and those tests completed

[edgao]

weird. That feels like a coincidence, destination-s3 doesn't even apply the airbyte-java-connector plugin...

(nw about the ping - I thought we used to run tests on draft PRs, but I guess not anymore)