Added readonly optimizations

cjmalloy · Aug 9, 2022 · 0ac10f3 · 0ac10f3
1 parent 6239836
commit 0ac10f3
Show file tree

Hide file tree

Showing 7 changed files with 20 additions and 0 deletions.
diff --git a/src/main/java/jasper/service/ExtService.java b/src/main/java/jasper/service/ExtService.java
@@ -63,12 +63,14 @@ public void create(Ext ext) {
 		extRepository.save(ext);
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadTag(#tag)")
 	public Ext get(String tag) {
 		return extRepository.findOneByQualifiedTag(tag)
 							.orElseThrow(() -> new NotFoundException("Ext " + tag));
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadQuery(#filter)")
 	public Page<Ext> page(TagFilter filter, Pageable pageable) {
 		return extRepository

diff --git a/src/main/java/jasper/service/FeedService.java b/src/main/java/jasper/service/FeedService.java
@@ -54,17 +54,20 @@ public void create(Feed feed) {
 		feedRepository.save(feed);
 	}
 
+	@Transactional(readOnly = true)
 	@PostAuthorize("@auth.canReadRef(returnObject)")
 	public FeedDto get(String url, String origin) {
 		var result = feedRepository.findOneByUrlAndOrigin(url, origin)
 								   .orElseThrow(() -> new NotFoundException("Feed " + origin + " " + url));
 		return mapper.domainToDto(result);
 	}
 
+	@Transactional(readOnly = true)
 	public boolean exists(String url, String origin) {
 		return feedRepository.existsByUrlAndOrigin(url, origin);
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadQuery(#filter)")
 	public Page<FeedDto> page(RefFilter filter, Pageable pageable) {
 		return feedRepository

diff --git a/src/main/java/jasper/service/OriginService.java b/src/main/java/jasper/service/OriginService.java
@@ -39,19 +39,22 @@ public void create(Origin origin) {
 		originRepository.save(origin);
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("hasRole('ADMIN')")
 	public Origin get(String origin) {
 		return originRepository.findById(origin)
 							   .orElseThrow(() -> new NotFoundException(origin));
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("hasRole('ADMIN')")
 	public Page<Origin> page(OriginFilter filter, Pageable pageable) {
 		return originRepository
 			.findAll(filter.spec(),
 				pageable);
 	}
 
+	@Transactional(readOnly = true)
 	public Page<OriginNameDto> pageNames(OriginFilter filter, Pageable pageable) {
 		return originRepository
 			.findAll(filter.spec(),

diff --git a/src/main/java/jasper/service/PluginService.java b/src/main/java/jasper/service/PluginService.java
@@ -34,17 +34,20 @@ public void create(Plugin plugin) {
 		pluginRepository.save(plugin);
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadTag(#tag)")
 	public Plugin get(String tag) {
 		return pluginRepository.findOneByQualifiedTag(tag)
 							   .orElseThrow(() -> new NotFoundException("Plugin " + tag));
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadTag(#tag)")
 	public boolean exists(String tag) {
 		return pluginRepository.existsByQualifiedTag(tag);
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadQuery(#filter)")
 	public Page<Plugin> page(TagFilter filter, Pageable pageable) {
 		return pluginRepository

diff --git a/src/main/java/jasper/service/RefService.java b/src/main/java/jasper/service/RefService.java
@@ -59,6 +59,7 @@ public void create(Ref ref) {
 		ingest.ingest(ref);
 	}
 
+	@Transactional(readOnly = true)
 	@PostAuthorize("@auth.canReadRef(returnObject)")
 	public RefDto get(String url, String origin) {
 		var result = refRepository.findOneByUrlAndOrigin(url, origin)
@@ -67,10 +68,12 @@ public RefDto get(String url, String origin) {
 		return mapper.domainToDto(result);
 	}
 
+	@Transactional(readOnly = true)
 	public boolean exists(String url, String origin) {
 		return refRepository.existsByUrlAndOrigin(url, origin);
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadQuery(#filter)")
 	public Page<RefDto> page(RefFilter filter, Pageable pageable) {
 		return refRepository
@@ -81,6 +84,7 @@ public Page<RefDto> page(RefFilter filter, Pageable pageable) {
 			.map(mapper::domainToDto);
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadQuery(#filter)")
 	public long count(RefFilter filter) {
 		return refRepository

diff --git a/src/main/java/jasper/service/TemplateService.java b/src/main/java/jasper/service/TemplateService.java
@@ -34,17 +34,20 @@ public void create(Template template) {
 		templateRepository.save(template);
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadTag(#tag)")
 	public Template get(String tag) {
 		return templateRepository.findOneByQualifiedTag(tag)
 								 .orElseThrow(() -> new NotFoundException("Template " + tag));
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadTag(#tag)")
 	public boolean exists(String tag) {
 		return templateRepository.existsByQualifiedTag(tag);
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadQuery(#filter)")
 	public Page<Template> page(TemplateFilter filter, Pageable pageable) {
 		return templateRepository.findAll(filter.spec(), pageable);

diff --git a/src/main/java/jasper/service/UserService.java b/src/main/java/jasper/service/UserService.java
@@ -35,13 +35,15 @@ public void create(User user) {
 		userRepository.save(user);
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadTag(#tag)")
 	public UserDto get(String tag) {
 		var result = userRepository.findOneByQualifiedTag(tag)
 								   .orElseThrow(() -> new NotFoundException("User " + tag));
 		return mapper.domainToDto(result);
 	}
 
+	@Transactional(readOnly = true)
 	@PreAuthorize("@auth.canReadQuery(#filter)")
 	public Page<UserDto> page(TagFilter filter, Pageable pageable) {
 		return userRepository