Browse Source

DATAJDBC-400 - Deprecated Identifier for those events that also have an entity.

Retrieval of the Identifier has been shown to cost significant performance.

Original pull request: #164.
pull/165/head
Jens Schauder 6 years ago committed by Mark Paluch
parent
commit
af3b9f8adb
  1. 8
      spring-data-relational/src/main/java/org/springframework/data/relational/core/mapping/event/RelationalEventWithId.java
  2. 6
      spring-data-relational/src/main/java/org/springframework/data/relational/core/mapping/event/SimpleRelationalEvent.java

8
spring-data-relational/src/main/java/org/springframework/data/relational/core/mapping/event/RelationalEventWithId.java

@ -39,11 +39,13 @@ public class RelationalEventWithId extends SimpleRelationalEvent implements With
this.id = id; this.id = id;
} }
/* /**
* (non-Javadoc) * Events with an identifier will always return a {@link Specified} one.
* @see org.springframework.data.jdbc.core.mapping.event.JdbcEvent#getId() *
* @deprecated obtain the id from the entity instead.
*/ */
@Override @Override
@Deprecated
public Specified getId() { public Specified getId() {
return id; return id;
} }

6
spring-data-relational/src/main/java/org/springframework/data/relational/core/mapping/event/SimpleRelationalEvent.java

@ -43,11 +43,11 @@ class SimpleRelationalEvent extends ApplicationEvent implements RelationalEvent
this.change = change; this.change = change;
} }
/* /**
* (non-Javadoc) * @deprecated obtain the id from the entity instead.
* @see org.springframework.data.jdbc.core.mapping.event.JdbcEvent#getId()
*/ */
@Override @Override
@Deprecated
public Identifier getId() { public Identifier getId() {
return (Identifier) getSource(); return (Identifier) getSource();
} }

Loading…
Cancel
Save