build: remove empty module backward-compat yahoo-custom-version (#4331)

### Changes

1. Remove empty module backward-compat yahoo-custom-version
2. fix a minor annotation in LedgerHandle

Signed-off-by: ZhangJian He <[email protected]>

Author: hezhangjian

.github/workflows/bk-ci.yml

@@ -329,7 +329,7 @@ jobs:
 
       - name: Other tests
         run: |
-          mvn -B -nsu -DbackwardCompatTests -pl :bc-non-fips,:hierarchical-ledger-manager,:hostname-bookieid,:old-cookie-new-cluster,:recovery-no-password,:upgrade-direct,:yahoo-custom-version test
+          mvn -B -nsu -DbackwardCompatTests -pl :bc-non-fips,:hierarchical-ledger-manager,:hostname-bookieid,:old-cookie-new-cluster,:recovery-no-password,:upgrade-direct test
 
       - name: Upload container logs on failure
         uses: actions/upload-artifact@v4

bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerHandle.java

@@ -2331,7 +2331,7 @@ List<BookieId> getCurrentEnsemble() {
 
     /**
      * Return a {@link WriteSet} suitable for reading a particular entry.
-     * This will include all bookies that are cotna
+     * This will include all bookies that are part of the ensemble for the entry.
      */
     WriteSet getWriteSetForReadOperation(long entryId) {
         if (stickyBookieIndex != STICKY_READ_BOOKIE_INDEX_UNSET) {

tests/backward-compat/pom.xml

@@ -35,7 +35,6 @@
     <module>recovery-no-password</module>
     <module>old-cookie-new-cluster</module>
     <module>current-server-old-clients</module>
-    <module>yahoo-custom-version</module>
     <module>bc-non-fips</module>
   </modules>
   <dependencyManagement>