Hibernate: Join vermeiden

[GilbertGrape]

Hallo,

folgende Situation:

Ich habe eine Klasse/Tabelle UnmodifiedText, die eine 1-n-Beziehung zu einer Verknüpfungstabelle UnmodifiedText_Paragraph hat. Das sieht ungefähr so aus:

[highlight=Java]public class UnmodifiedText {

@Id
@GeneratedValue(strategy = GenerationType.AUTO)
private Long id;
...
...
@OneToMany(mappedBy = "unmodifiedText", fetch=FetchType.EAGER)
@Cascade({org.hibernate.annotations.CascadeType.DELETE})
private Set<UnmodifiedText_Paragraph> utp = new HashSet<UnmodifiedText_Paragraph>();
[/highlight]

Nun hätte ich gern die Texte, die noch kein UnmodifiedText_Paragraph haben, also zu denen es in der Verknüpfungstabelle noch keinen Eintrag gibt.
Im Moment mach ich das mit einem Join so:
[highlight=Java] query="from UnmodifiedText as t left join fetch t.utp as u where u.unmodifiedText is null"
[/highlight]

Das dauert recht lange, deshalb wollte ich fragen, ob das auch irgendwie anders geht. Ich habs so versucht:
[highlight=Java]query="from UnmodifiedText as t where t.utp is null"[/highlight]
aber das geht nicht. Da kommt dann ne Exception.
Ich habs auch mit "not exists in..." versucht, aber das is auch langsam.

Gibts da ne Möglichkeit?

Gruß, Gilbert

 

[KSG9|sebastian]

Hallo,

folgende Situation:

Ich habe eine Klasse/Tabelle UnmodifiedText, die eine 1-n-Beziehung zu einer Verknüpfungstabelle UnmodifiedText_Paragraph hat. Das sieht ungefähr so aus:

[highlight=Java]public class UnmodifiedText {

@Id
@GeneratedValue(strategy = GenerationType.AUTO)
private Long id;
...
...
@OneToMany(mappedBy = "unmodifiedText", fetch=FetchType.EAGER)
@Cascade({org.hibernate.annotations.CascadeType.DELETE})
private Set<UnmodifiedText_Paragraph> utp = new HashSet<UnmodifiedText_Paragraph>();
[/highlight]

Nun hätte ich gern die Texte, die noch kein UnmodifiedText_Paragraph haben, also zu denen es in der Verknüpfungstabelle noch keinen Eintrag gibt.
Im Moment mach ich das mit einem Join so:
[highlight=Java] query="from UnmodifiedText as t left join fetch t.utp as u where u.unmodifiedText is null"
[/highlight]

Das dauert recht lange, deshalb wollte ich fragen, ob das auch irgendwie anders geht. Ich habs so versucht:
[highlight=Java]query="from UnmodifiedText as t where t.utp is null"[/highlight]
aber das geht nicht. Da kommt dann ne Exception.
Ich habs auch mit "not exists in..." versucht, aber das is auch langsam.

Gibts da ne Möglichkeit?

Gruß, Gilbert

Wenn das lange Dauert dann würde ich an deiner Stelle mal prüfen ob auf den entsprechenden Feldern ein Index angelegt ist.
Aber wie sollte das anders gehen?

1:n bedeutet dass das "n-Obekt" eine Spalte hat mit dem Wert des Primärschlüssels vom "1-Objekt", Beispiel
Person - Konto : 1:n

Person: Person_ID (PK), Name, Geburtsdatum
Konto: Konto_ID( PK), Person_ID (FK), Kontonummer

Das wiederum bedeutet: Um rauszufinden ob Person 1 ein Konto hat musst du prüfen ob in der Kontotabelle ein Satz steht welcher Person_ID=1 hat. Und um das zu tun musst du entweder per Join oder per Where einschränken.

Und dass das langsam ist kann ich mir irgendwie nicht vorstellen. Wir haben Tabellen mit Millionen von Sätzen und selbst dort ist ein so einfacher Join welcher über einen (Unique)Index geht verdammt schnell.

 

[KSG9|sebastian]

Ach ja, du kannst z.B. sowas verwenden:

String hql = "from Person p where size(p.address) > 0";

oder sowas müsst auch gehen:

String hql = "from Person p where p.address.id is null";

Allerdings wird das keinen Performancevorteil bringen. Variante 1 macht wahrscheinlich ne count()-Abfrage welche auf jeden Fall langsamer ist. Variante 2 wird auch nen Join machen.
Also dein Problem dürfte weniger am Query und mehr an der Tabelle/Datenbank liegen 🙂

 

[The_S]

Was Hibernate da teilweise generiert ist sowieso recht abenteuerlich 😉

 

[KSG9|sebastian]

Was Hibernate da teilweise generiert ist sowieso recht abenteuerlich 😉

Wieso?

 

[The_S]

Hab jetzt grad kein konkretes Beispiel bereit, aber letztens hat Hibernate z. B. so etwas gebaut (stark vereinfacht):

[HIGHLIGHT="SQL"]Select * from A a
Inner Join A b ON a.id = b.id[/HIGHLIGHT]

Natürlich ohne, dass so etwas von mir beabsichtigt gewesen wäre. Aber ne Maschine kann natürlich nicht automatisiert so gute SQLs bauen, wie ein denkender Mensch, der etwas von der Materie versteht.

 

[GilbertGrape]

Vielen Dank erstmal!
Also in der ersten Tabelle (UnmodifiedText) hab ich explizit einen Index angelegt und für alle anderen Tabellen wurde (automatisch von Hibernate?) ein Index auf die ID-Spalte angelegt. (jedenfalls zeigt das mein DB-Developer an)
Ich les ja auch nicht alles aufeinmal aus. Jedenfalls begrenze ich durch setMaxResult, aber das macht bei dieser Abfrage irgendwie keinen Unterschied.

 

[KSG9|sebastian]

Ich wüsste nicht das Hibernate verpfuschte SQLs baut, aber okay 🙂

@GilbertGrape:

Es sollte jeweils einen Index für die Join-Spalte geben.
Also in der Parenttabelle auf dem Primärschlüssel und in der referenzierten Tabelle auf dem Fremdschlüssel.

Was erzeugt denn Hibernate für nen SQL? Mach mal hibernate.show_sql auf true in der config.
Wie gesagt, dass die Abfrage langsam ist kann ich mir net vorstellen und es gibt imho auch keinen anderen Weg das so zu selektieren. Ein Join ist da wohl das performanteste Mittel.
Nimm mal die Abfrage wie sie Hibernate ausführt und lass mit ner entsprechenden Software (z.B. Visual Explain für DB2, DBVisualizer kanns glaub auch) nen Zugriffsplan erstellen, dann siehst du recht schnell ob ein kompletter Tablescan gemacht wird oder ob über den Indexbaum zugegriffen wird.

 

[The_S]

Ich wüsste nicht das Hibernate verpfuschte SQLs baut, aber okay 🙂

Funktionieren ja auch alle. Nur sind nicht alle gerade optimal (von der Performance) umgesetzt.

 

[GilbertGrape]

Das SQL von Hibernate sieht dann so aus:

select
unmodified0_.id as id4_0_,
utp1_.paragraph as paragraph6_1_,
utp1_.unmodifiedText as unmodifi2_6_1_,
unmodified0_.creation as creation4_0_,
unmodified0_.HASH_VALUE as HASH3_4_0_,
unmodified0_.lastUpdate as lastUpdate4_0_,
unmodified0_.text as text4_0_,
unmodified0_.text_id as text6_4_0_,
unmodified0_.type as type4_0_,
utp1_.position as position6_1_,
utp1_.unmodifiedText as unmodifi2_0__,
utp1_.paragraph as paragraph0__
from
UNMODIFIED_TEXT unmodified0_
left outer join
UnmodifiedText_Paragraph utp1_
on unmodified0_.id=utp1_.unmodifiedText
where
utp1_.unmodifiedText is null

Das scheint schon so in Ordnung.
Indizes haben auch alle Primärschlüssel (daran mach ich den Join).
Und diese Abfrage soll ich jetzt mit diesem DBVisualizer überprüfen??

 

[KSG9|sebastian]

Jenachdem was du für ne Datenbank hast gibt's sicher ein Tool mit dem du nen Zugriffsplan erstellen kannst.
Mit was arbeitest du denn?

 

[GilbertGrape]

oracle (ich glaub, das is 11g)

 

[KSG9|sebastian]

Dann hast du doch sicher den Oracle SQL-Developer? Da kann man mit Sicherheit auch Zugriffspläne für nen Select zeigen.
Musst mal schauen ob du in der Doku was findest oder so.