KAFKA-2423: Introduce Scalastyle#560
Conversation
|
Getting the discussion stared on ScalaStyle. This pull request only contains the integration to gradle and the configuration of the proposed scalastyle rules. Once this is reviewed and the rules are agreed upon and I can add a commit with the changes to make the build pass and reduce the noise from some of the ones we want to enable but can't yet. |
|
@ijuma Any feedback on the proposed rules? |
Just the buildscript changes and rules configuration.
c164bf0 to
4c8ef9a
Compare
|
@granthenke I haven't had a chance to look at them in detail and it probably won't happen before next week. |
|
@ijuma Thanks for the update! |
|
@granthenke How is this PR related to #284? BTW would you like to continue work on either one of these two? |
|
Refer to this link for build results (access rights to CI server needed): |
|
Refer to this link for build results (access rights to CI server needed): |
|
Refer to this link for build results (access rights to CI server needed): |
…tart a new execution. (apache#560)
|
Hi @granthenke , It seems like this PR stalled. I'll close it out for now, but if you or anyone else want to resume this work, please feel free to re-open it (or start a new one)! Thanks, |
…pache#560) Add startup validation that enforces the dependency chain between diskless feature flags: diskless.storage.system.enable → diskless.managed.rf.enable → diskless.allow.from.classic.enable (+ remote.log.storage.system.enable) Brokers now fail fast with a clear error message if flags are enabled without their prerequisites. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
5 participants
Just the buildscript changes and rules configuration.