Skip to content

Build update/cleanup#102

Open
ctubbsii wants to merge 1 commit intoapache:mainfrom
ctubbsii:build-cleanup-for-beta2
Open

Build update/cleanup#102
ctubbsii wants to merge 1 commit intoapache:mainfrom
ctubbsii:build-cleanup-for-beta2

Conversation

@ctubbsii
Copy link
Member

@ctubbsii ctubbsii commented Mar 7, 2026

  • Fix execution of QA plugins, such as license-maven-plugin
  • Add generated code to the build in a simpler way without checking it in, or subjecting it to QA checks
  • Place version management in parent pom
  • Organize child modules in modules/ directory
  • Use symlinks for test files instead of copying them with the resources plugin
  • Configure annotation processor library explicitly instead of placing on the classpath
  • Use fully-qualified module names (org.apache.*) to match the convention established by the automatic module names in the main Accumulo repo
  • Address spotbugs, Java 21+ compiler warnings, and errorprone issues
  • Update parent POM and plugins
  • Add GitHub issue templates and fixup workflows

* Fix execution of QA plugins, such as license-maven-plugin
* Add generated code to the build in a simpler way without checking it
  in, or subjecting it to QA checks
* Place version management in parent pom
* Organize child modules in modules/ directory
* Use symlinks for test files instead of copying them with the resources
  plugin
* Configure annotation processor library explicitly instead of placing
  on the classpath
* Use fully-qualified module names (org.apache.*) to match the
  convention established by the automatic module names in the main
  Accumulo repo
* Address spotbugs, Java 21+ compiler warnings, and errorprone issues
* Update parent POM and plugins
* Add GitHub issue templates and fixup workflows
@ctubbsii ctubbsii added this to the 1.0.0 milestone Mar 7, 2026
@ctubbsii ctubbsii self-assigned this Mar 7, 2026
@ctubbsii
Copy link
Member Author

ctubbsii commented Mar 7, 2026

I noticed while working on this that we have a lot of @since 1.0.0 tags. I'm not sure it's useful to have those. If a method or class has always existed, since 1.0.0, there isn't really much information being added by these. And, they trigger errorprone if the javadoc exists for that annotation only, without a summary sentence.

@ctubbsii
Copy link
Member Author

ctubbsii commented Mar 7, 2026

Benchmarks take 28 minutes in my dev environment with these changes.

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.

1 participant