Skip to content

Commit 8a47b8e

Browse files
committed
[temp] Disable eager serialization of XQuery results over XML:RPC as the result format is incompatible with eXist-db 6.x.x
This could be enabled later in Elemental for improved serialization over XML-RPC and performance, but doing so would compromise the current 100% XML:RPC API compatibility with eXist-db 6.x.x.
1 parent 71d45aa commit 8a47b8e

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

exist-core/src/main/java/org/exist/xmlrpc/RpcConnection.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2028,7 +2028,10 @@ private Map<String, Object> queryResultToTypedRpcResponse(final long startTime,
20282028
result.put("docUri", "temp_xquery/" + item.hashCode());
20292029
result.put("nodeId", String.valueOf(ni.getNodeNumber()));
20302030
result.put("xdmSerializationOptions", xdmSerializationOptions);
2031-
result.put("value", item); // NOTE(AR) In-memory Nodes are transient and so need to be sent directly in the result
2031+
2032+
// TODO(AR) enable this later in Elemental for improved serialization over XML-RPC and performance (it also means that we could then cleanup the XML:DB Remote API by removing its uses of XQJ Marshaller and org.exist.xquery.value.*)
2033+
// NOTE(AR) In-memory Nodes can alternatively be returned directly as part of the result, however doing so changes the XML:RPC result format in a manner that would be incompatible with eXist-db 6.x.x
2034+
// result.put("value", item);
20322035

20332036
} else {
20342037
LOG.error("Omitting from results, unsure how to process: {}", item.getClass());

0 commit comments

Comments
 (0)