Elgato_dark/OpenMetadata
1
0
Fork 0
mirror of https://github.com/open-metadata/OpenMetadata synced 2026-05-24 09:39:11 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

fix(rdf): make buildPredicateInList public so JenaFusekiStorage can use it

Browse source 
JenaFusekiStorage (org.openmetadata.service.rdf.storage) lives in a different
package than RdfRepository (org.openmetadata.service.rdf), so the
package-private buildPredicateInList helper introduced in 857c09 couldn't be
called from JenaFusekiStorage.bulkStoreRelationships — CI was failing with:

  [ERROR] JenaFusekiStorage.java:[606,51] buildPredicateInList(Set<String>)
  is not public in RdfRepository; cannot be accessed from outside package

Promote it to public alongside RELATIONSHIP_HOOK_PREDICATES (which is the
only data this helper renders) so the cross-package call resolves. Local
javac across the touched RDF files now reports zero new errors; the only
remaining build failures are the pre-existing es.co.elastic.clients shading
issues unrelated to this PR.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Sriharsha Chintalapani 2026-05-15 10:21:42 -07:00
parent 4242c154be
commit 03c5d4f6dc
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
openmetadata-service/src/main/java/org/openmetadata/service/rdf/RdfRepository.java
View file

@ -430,7 +430,9 @@ public class RdfRepository {
}
// Build a comma-separated "<uri1>, <uri2>, ..." for SPARQL `?p IN (...)` lists.
static String buildPredicateInList(Set<String> uris) {
// public so JenaFusekiStorage (in storage subpackage) can reuse the same
// RELATIONSHIP_HOOK_PREDICATES rendering for its per-source DELETE filter.
public static String buildPredicateInList(Set<String> uris) {
StringBuilder sb = new StringBuilder();
boolean first = true;
for (String uri : uris) {