Почему hibernate выполняет два запроса для активной загрузки двунаправленной ассоциации @OneToOne? - PullRequest
Новая
       23

Почему hibernate выполняет два запроса для активной загрузки двунаправленной ассоциации @OneToOne?

9 голосов
/ 01 августа 2010

у меня есть сущность A, которая имеет -B сущность, а B имеет -A A с двунаправленной ассоциацией @OneToOne.

Теперь, когда я нахожу все записи A, hibernate выполняет два запроса с левым внешним соединением наB, что-то вроде этого: 

select a.id, a.id_b, a.field1, b.id, b.field1 from A as a, B as b left outer join b ON b.id=a.id_b;
select a.id, a.id_b, a.field1, b.id, b.field1 from A as a, B as b left outer join b ON b.id=a.id_b WHERE b.id=?

Первый запрос загружает поля A и B, и это нормально, но зачем выполнять второй запрос для перезагрузки A?Я думаю, что этот запрос загружает содержимое A в B, но этот A, очевидно, является A, который содержит B ... поэтому он уже загружен первым запросом, не так ли?

- EDIT --

Объект A: 

@Entity
public class A implements Serializable{
    // id and other ecc ecc
    @OneToOne
    @JoinColumn(name="id_b")
    B b;
}

Объект B: 

@Entity
public class B implements Serializable{
    // id and other ecc ecc
    @OneToOne(mappedBy="b")
    A a;
}

Это ситуация, и findAll для A требует два запроса ...почему?

Ответы [ 2 ]

6 голосов
/ 02 августа 2010

Удар, если A и B совместно используют один и тот же столбец первичного ключа , где оба объекта объединены с использованием их первичного ключа , вместо этого следует использовать @PrimaryKeyJoinColumn 

@Entity
public class A implements Serializable {

    private MutableInt id = new MutableInt();

    private B b;

    public void setIdAsMutableInt(MutableInt id) {
        this.id = id;
    }

    @Id
    @GeneratedValue
    public Integer getId() {
        return id.intValue();
    }

    public void setId(Integer id) {
        this.id.setValue(id);
    }

    /**
      * Any ToOne annotation, such as @OneToOne and @ManyToOne, is EARGELY loaded, by default
      */
    @OneToOne(fetch=FetchType.LAZY)
    @PrimaryKeyJoinColumn
    @Cascade(CascadeType.SAVE_UPDATE)
    public B getB() {
        return b;
    }

    public void setB(B b) {
        b.setIdAsMutableInt(id);

        this.b = b;
    }

}

И B Обратите внимание вам не нужен атрибут mappedBy из-за @ PrimaryKeyJoinColumn 

@Entity
public class B implements Serializable {

    private MutableInt id = new MutableInt();

    private A a;

    public void setIdAsMutableInt(MutableInt id) {
        this.id = id;
    }

    @Id
    public Integer getId() {
        return id.intValue();
    }

    public void setId(Integer id) {
        this.id.setValue(id);
    }

    @OneToOne(fetch=FetchType.LAZY)
    @PrimaryKeyJoinColumn
    public A getA() {
        return a;
    }

    public void setA(A a) {
        this.a = a;
    }

}

Давайте проверим (вы можете проверить, если хотите) 

A a = new A();
B b = new B();

a.setB(b);

/**
  * b property will be saved because Cascade.SAVE_UPDATE
  */
Serializable id = session.save(a);

b = (B) session
        .createQuery("from B b left join fetch b.a where b.id = :id")
        .setParameter("id", id)
        .list()
        .get(0);

Assert.assertEquals(b.getId(), b.getA().getId());

Обратите внимание, что я использую поле MutableInt ( инкапсулировано свойством Integer) вместо Integer, потому что Integer является неизменяемым типом, так как A и B совместно используют один и тот же назначенный идентификатор

Но если A и B объединены с использованием другого, отличного от их первичного ключа , вы должны использовать @JoinColumn и mappedBy (двунаправленное отношение, справа) следующим образом 

@Entity
public class A implements Serializable {

    private Integer id;

    private B b;

    @Id
    @GeneratedValue
    public Integer getId() {
        return id;
    }

    public void setId(Integer id) {
        this.id = id;
    }

    /**
      * mappedBy="a" means: Look at "a" field / property at B Entity. If it has any assigned value, join us Through B_ID foreign key column
      */
    @OneToOne(fetch=FetchType.LAZY, mappedBy="a")
    /**
      * Table A has a foreign key column called "B_ID"
      */ 
    @JoinColumn(name="B_ID")
    @Cascade(CascadeType.SAVE_UPDATE)
    public B getB() {
        return b;
    }

    public void setB(B b) {
        this.b = b;
    }

}

И B 

@Entity
public class B implements Serializable {

    private Integer id;

    private A a;

    public void setIdAsMutableInt(MutableInt id) {
        this.id = id;
    }

    @Id
    @GeneratedValue
    public Integer getId() {
        return id;
    }

    public void setId(Integer id) {
        this.id = id;
    }

    @OneToOne(fetch=FetchType.LAZY)
    public A getA() {
        return a;
    }

    public void setA(A a) {
        this.a = a;
    }

}

Для проверки 

A a = new A();
B b = new B();

/**
  * Set up both sides
  * Or use some kind of add convenience method
  */ 
a.setB(b);
b.setA(a);

/**
  * b property will be saved because Cascade.SAVE_UPDATE
  */
Serializable id = session.save(a);

b = (B) session
        .createQuery("from B b left join fetch b.a where b.id = :id")
        .setParameter("id", id)
        .list()
        .get(0);

Используя сторону владельца B, вы получите два оператора выбора Это происходит потому, что BТаблица не содержит столбца внешнего ключа, который указывает на таблицу A Но, используя

"из A, левое соединение извлекает ab, где a.id =: id"

Вы получите только один оператор выбора потому что A знает, как извлечь объединенное B, используя столбец внешнего ключа B_ID

0 голосов
/ 01 августа 2010

Как выглядит ваше отображение?

Правильно ли реализованы классы A и B hashCode() и equals(), чтобы Hibernate мог определить, что экземпляр A, на который указывает B, является тем же экземпляром первого A?

Звучит так, будто вы пытаетесь смоделировать двунаправленное взаимно-однозначное сопоставление - взгляните на раздел руководства по этому , чтобы увидеть рекомендуемые методы его выполнения.

...