search

etiennestuder / gradle-jooq-plugin

Gradle plugin that integrates jOOQ.
Apache License 2.0
512 stars 84 forks source link

#625 Prevent race condition when checking if inputs are up-to-date #626

Closed lared closed 1 year ago

lared commented 1 year ago

Related to #625

I know that this way of solving concurrency issues is rather low effort, but the alternative is to create separate message digests to reduce contention, which creates slightly more waste in case of non-parallel multi-project builds. I'm open to adjusting to whatever you prefer.

I don't think this solution (or any other, really) is reasonably testable either, so I'm not including any test. I was no longer able to reproduce #625 manually after introducing this change.

etiennestuder commented 1 year ago

Thank you for the PR. @lared !