Skip to content

Commit fb374f0

Browse files
duncdrumclaude
andcommitted
[doc] Update known-build-issues docs and pom comment
AGENTS.md: MoveResourceTest no longer hangs after the HTTP/1.1 fix — remove it from the warning. Document the two permanently excluded tests (FragmentsTest, ConcurrencyTest) with their root causes. exist-core/pom.xml: FragmentsTest comment previously claimed the test passes locally in ~54s. Confirmed to hang locally as well (killed after 12 min); root cause is not fully resolved by the isShuttingDownOrDown fix alone. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent ad9b932 commit fb374f0

3 files changed

Lines changed: 7 additions & 3 deletions

File tree

AGENTS.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,9 @@ docker run -d --name existdb -p 8080:8080 -p 8443:8443 existdb/existdb:local
9898

9999
### Known build issues
100100

101-
- Full test suite can hang on flaky infrastructure tests (`MoveResourceTest`, `RenameCollectionTest`). Check with `jstack` and kill if stuck >15 min.
101+
- Full test suite can hang on flaky infrastructure tests (`RenameCollectionTest`). Check with `jstack` and kill if stuck >15 min.
102102
- `RenameCollectionTest` "Connection refused" failures are pre-existing and unrelated to XQuery changes.
103+
- `org.exist.xmldb.concurrent.FragmentsTest` and `org.exist.collections.ConcurrencyTest` are excluded from the surefire run (see `exist-core/pom.xml`). `FragmentsTest` hangs during BrokerPool shutdown even locally; `ConcurrencyTest` can deadlock under concurrent collection access.
103104

104105
## Parser (ANTLR 2)
105106

exist-core/pom.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1216,8 +1216,8 @@ The BaseX Team. The original license statement is also included below.]]></pream
12161216
see https://github.com/eXist-db/exist/issues/3685 -->
12171217
<exclude>org.exist.collections.ConcurrencyTest</exclude>
12181218

1219-
<!-- Passes locally (~54s) but hangs on CI during BrokerPool shutdown,
1220-
causing the entire unit test step to time out.
1219+
<!-- Hangs both locally and on CI during BrokerPool shutdown;
1220+
root cause not fully resolved (isShuttingDownOrDown fix was insufficient).
12211221
see https://github.com/eXist-db/exist/pull/6153 -->
12221222
<exclude>org.exist.xmldb.concurrent.FragmentsTest</exclude>
12231223
</excludes>

exist-core/src/test/java/org/exist/xquery/XQueryUpdateTest.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -515,6 +515,9 @@ public void removeTestData() throws EXistException, PermissionDeniedException, I
515515
}
516516

517517

518+
// Precondition: caller must hold an active DBBroker on the current thread (e.g. via withBroker).
519+
// Txn.close() calls pool.getBroker() to remove the transaction; without an active broker on the
520+
// thread it may draw a different instance and throw IllegalStateException.
518521
private void store(DBBroker broker, String docName, String data) throws PermissionDeniedException, EXistException, SAXException, LockException, IOException {
519522
Collection root;
520523
final BrokerPool pool = existEmbeddedServer.getBrokerPool();

0 commit comments

Comments
 (0)