Question

我正在分析我的应用程序的性能，我注意到以下内容。我有这个问题：

public List<Rol> findAll() {
        return mySessionFactory.getCurrentSession().createQuery("from Rol").list();
    }

这会返回此查询：

SELECT rol0_._id as column1_2_, rol0_.description as descripc2_2_, rol0_.name as nombre6_2_, rol0_.status as status7_2_ FROM rol rol0_

这很好，但在我的jsp中我有以下内容：

<form:select multiple="true" path="roles" required="true">
                    <form:options items="${roles}" itemValue="id" itemLabel="nombre" />
                </form:select>

这会为每个rol创建一个新查询：

DEBUG: org.hibernate.SQL - select rol0_._id as column1_2_0_, rol0_.description as descripc2_2_0_, rol0_.name as name6_2_0_, rol0_.status as status7_2_0_ from rol rol0_ where rol0_._id=?
TRACE: org.hibernate.type.descriptor.sql.BasicBinder - binding parameter [1] as [VARCHAR] - 1
DEBUG: org.hibernate.SQL - select rol0_._id as column1_2_0_, rol0_.description as descripc2_2_0_, rol0_.name as name6_2_0_, rol0_.status as status7_2_0_ from rol rol0_ where rol0_._id=?
TRACE: org.hibernate.type.descriptor.sql.BasicBinder - binding parameter [1] as [VARCHAR] - 2
DEBUG: org.hibernate.SQL - select rol0_._id as column1_2_0_, rol0_.description as descripc2_2_0_, rol0_.name as name_2_0_, rol0_.status as status7_2_0_ from rol rol0_ where rol0_._id=?
TRACE: org.hibernate.type.descriptor.sql.BasicBinder - binding parameter [1] as [VARCHAR] - 20130915150706256
DEBUG: org.hibernate.SQL - select rol0_._id as column1_2_0_, rol0_.description as descripc2_2_0_, rol0_.name as name6_2_0_, rol0_.status as status7_2_0_ from rol rol0_ where rol0_._id=?
TRACE: org.hibernate.type.descriptor.sql.BasicBinder - binding parameter [1] as [VARCHAR] - 3

在我的控制器中我有：

List<Rol> roles = rolDao.findAll();
ModelAndView mav = new ModelAndView("usuarioNew");
mav.getModelMap().put("roles", roles);

有什么方法可以避免这种情况吗？第一个查询包含我需要的所有信息，我不想要额外的查询。

修改

@Entity
@Table(name = "rol", uniqueConstraints = { @UniqueConstraint(columnNames = "name") })
public class Rol implements Serializable{

    @Id
    @Column(name = "_id")
    private String id;

    @Column(name = "name")
    @NotEmpty
    private String name;

    @Column(name = "description")
    private String description;

    @Column(name = "status")
    private String status;


    @NotEmpty
    @Fetch(FetchMode.JOIN)
    @OneToMany(mappedBy = "rolPermission_pk.rol", orphanRemoval = true, cascade=CascadeType.ALL)
    private Set<Rol_Permission> permissions = new HashSet<Rol_Permission>(0);

    ....//getters, setters
    }

EDIT2：

public class UserRolCollectionEditor extends CustomCollectionEditor {

     private final RolDAO rolDao;


    public UserRolCollectionEditor (Class<?> collectionType, RolDAO rolDao) {

        super(collectionType);
        this.rolDao = rolDao;

    }

    @Override
    protected Object convertElement(Object element) {
        String rolId = (String) element;

        Rol rol = rolDao.findById(rolId);
        User_Rol usuRol = new User_Rol();


        User user = new User();

        usuRol.setUser(user);
        usuRol.setRol(rol);
        usuRol.setStatus("active");

        return usuRol;
    }

}

EDIT3：

我做了一些测试，问题出在UserRolCollectionEditor和@Fetch（FetchMode.JOIN）上。如果我从代码中删除UserRolCollectionEditor和@Fetch（FetchMode.JOIN），我会得到一个查询。如果只删除@Fetch（FetchMode.JOIN），我会得到额外的查询。如果只删除UserRolCollectionEditor我得到额外的查询..我不知道该怎么办..

Answer 1

我不能说为什么hibernate会生成过多的查询，但是在JSP中访问$ {roles}时会因为存在持久性上下文（即开放会话）而创建它们。为了防止这种行为，您可以尝试事先关闭会话（我认为这更像是一种解决方法，而不是您案例中的解决方案）。有几种方法可以实现（使用每个请求的会话模式）：

在DAO中使用mySessionFactory.openSession（）和mySessionFactory.closeSession（）;
在DAO中使用getCurrentSession（）。beginTransaction（）和getCurrentSession（）。commitTransaction（）;
创建服务层，让Spring管理交易。

Great spring layered webapp example

Dealing with sessions and transactions in Hibernate

<强>更新

如果您在基础UserRolCollectionEditor中使用Rol rol = rolDao.findById(rolId);，则session.get()行rolDao可以创建选择查询。如果是这种情况，您可以将其更改为session.load()，以避免额外查询。简而言之，session.load()的常见使用场景是在实体对象之间创建关联 - 这正是您在UserRolCollectionEditor.convertElement()方法中所做的。

Short article shows difference between get and load

Hibernate性能：几个查询而不是一个表单之后：在jsp中选择

1 个答案: