[fix][doc] Remove lombok plugin and define FunctionRecordBuilder explicitly

[tisonkun]

• This refers to [improve][build] Add lombok plugin to solve Javadoc issue. #18520
• This refers to [improve][function] Support Record<?> as Function output type #16041

Verified locally with:

mvn -pl pulsar-client-api javadoc:javadoc 
mvn -pl pulsar-client-admin-api javadoc:javadoc
cd ./pulsar-functions 
mvn -pl api-java javadoc:javadoc

... results:

➜  pulsar git:(fix-javadoc) ls pulsar-client-api/target/site/apidocs
allclasses-index.html   jquery-ui.overrides.css resources
allpackages-index.html  legal                   script-dir
constant-values.html    member-search-index.js  script.js
deprecated-list.html    module-search-index.js  search.js
element-list            org                     serialized-form.html
help-doc.html           overview-summary.html   stylesheet.css
index-all.html          overview-tree.html      tag-search-index.js
index.html              package-search-index.js type-search-index.js
➜  pulsar git:(fix-javadoc) ls pulsar-client-admin-api/target/site/apidocs 
allclasses-index.html   jquery-ui.overrides.css resources
allpackages-index.html  legal                   script-dir
constant-values.html    member-search-index.js  script.js
deprecated-list.html    module-search-index.js  search.js
element-list            org                     serialized-form.html
help-doc.html           overview-summary.html   stylesheet.css
index-all.html          overview-tree.html      tag-search-index.js
index.html              package-search-index.js type-search-index.js
➜  pulsar git:(fix-javadoc) ls pulsar-functions/api-java/target/site/apidocs 
allclasses-index.html   legal                   script-dir
allpackages-index.html  member-search-index.js  script.js
deprecated-list.html    module-search-index.js  search.js
element-list            org                     stylesheet.css
help-doc.html           overview-summary.html   tag-search-index.js
index-all.html          overview-tree.html      type-search-index.js
index.html              package-search-index.js
jquery-ui.overrides.css resources
➜  pulsar git:(fix-javadoc) 

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

• Dependencies (add or upgrade a dependency)
• The public API
• The schema
• The default values of configurations
• The threading model
• The binary protocol
• The REST endpoints
• The admin CLI options
• The metrics
• Anything that affects deployment

Documentation

• doc
• doc-required
• doc-not-needed
• doc-complete

Matching PR in forked repository

PR in forked repository:

[codecov-commenter]

Codecov Report

Merging #19202 (1780e84) into master (4b0dc9a) will decrease coverage by 7.12%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #19202      +/-   ##
============================================
- Coverage     48.96%   41.84%   -7.13%     
+ Complexity     7300     2433    -4867     
============================================
  Files           424      235     -189     
  Lines         45473    16701   -28772     
  Branches       4672     1822    -2850     
============================================
- Hits          22268     6989   -15279     
+ Misses        20698     8959   -11739     
+ Partials       2507      753    -1754     

Flag	Coverage Δ
unittests	41.84% <ø> (-7.13%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...rcegroup/ResourceGroupNamespaceConfigListener.java
...e/pulsar/broker/service/EntryBatchIndexesAcks.java
...ce/ConsistentHashingStickyKeyConsumerSelector.java
...ker/service/plugin/EntryFilterWithClassLoader.java
...ker/loadbalance/impl/SimpleLoadCalculatorImpl.java
...ransactionBufferSnapshotBaseSystemTopicClient.java
...ache/pulsar/broker/loadbalance/ServiceRequest.java
...roker/admin/impl/OffloaderObjectsScannerUtils.java
...er/validator/TransactionBatchedWriteValidator.java
...oadbalance/extensions/data/TopBundlesLoadData.java
... and 597 more

[eolivelli]

Interesting!

+1

[tisonkun]

delombok should help, but then we should move the related class to src/main/lombok and it's the same level of change as define explicitly. I choose a way more understandable for me to follow.