Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Difference between CrudRepository findOne() and JpaRepository getOne()

I read that getOne() is lazy loaded and findOne() fetches the whole entity right away. I've checked the debugging log and I even enabled monitoring on my sql server to see what statements gets executed, I found that both getOne() and findOne() generates and executes the same query. However when I use getOne() the values are initially null (except for the id of course).

So could anyone please tell me, if both methods executes the same query on the database, why should I use one over the other? I'm basically looking for a way to fetch an entity without getting all of its children/attributes.

EDIT1:

Entity code

Dao code:

@Repository
public interface FlightDao extends JpaRepository<Flight, Long> {
}

Debugging log findOne() vs getOne()

EDIT2:

Thanks to Chlebik I was able to identify the problem. Like Chlebik stated, if you try to access any property of the entity fetched by getOne() the full query will be executed. In my case, I was checking the behavior while debugging, moving one line at a time, I totally forgot that while debugging the IDE tries to access object properties for debugging purposes (or at least that's what I think is happening), so debugging triggers the full query execution. I stopped debugging and then checked the logs and everything appears to be normal.

getOne() vs findOne() (This log is taken from MySQL general_log and not hibernate.

Debugging log

No debugging log

like image 569
prettyvoid Avatar asked Oct 19 '15 15:10

prettyvoid


2 Answers

It is just a guess but in 'pure JPA' there is a method of EntityManager called getReference. And it is designed to retrieve entity with only ID in it. Its use was mostly for indicating reference existed without the need to retrieve whole entity. Maybe the code will tell more:

// em is EntityManager
Department dept = em.getReference(Department.class, 30);  // Gets only     entity with ID property, rest is null
Employee emp = new Employee();
emp.setId(53);
emp.setName("Peter");
emp.setDepartment(dept);
dept.getEmployees().add(emp);
em.persist(emp);

I assume then getOne serves the same purpose. Why the queries generated are the same you ask? Well, AFAIR in JPA bible - Pro JPA2 by Mike Keith and Merrick Schincariol - almost every paragraph contains something like 'the behaviour depends on the vendor'.

EDIT:

I've set my own setup. Finally I came to conclusion that if You in any way interfere with entity fetched with getOne (even go for entity.getId()) it causes SQL to be executed. Although if You are using it only to create proxy (eg. for relationship indicator like shown in a code above), nothing happens and there is no additional SQL executed. So I assume in your service class You do something with this entity (use getter, log something) and that is why the output of these two methods looks the same.

ChlebikGitHub with example code
SO helpful question #1
SO helpful question #2

like image 113
Chlebik Avatar answered Oct 18 '22 13:10

Chlebik


Suppose you want to remove an Entity by id. In SQL you can execute a query like this :

"delete form TABLE_NAME where id = ?". 

And in Hibernate, first you have to get a managed instance of your Entity and then pass it to EntityManager.remove method.

Entity a = em.find(Entity.class, id);
em.remove(a);

But this way, You have to fetch the Entity you want to delete from database before deletion. Is that really necessary ?

The method EntityManager.getReference returns a Hibernate proxy without querying the database and setting the properties of your entity. Unless you try to get properties of the returned proxy yourself.

Method JpaRepository.getOne uses EntityManager.getReference method instead of EntityManager.find method. so whenever you need a managed object but you don't really need to query database for that, it's better to use JpaRepostory.getOne method to eliminate the unnecessary query.

like image 30
chubock Avatar answered Oct 18 '22 11:10

chubock