지금까지 배운 것을 바탕으로 성능을 개선해보자.
동기적으로 작성된 코드를 비동기적으로 변경하는 것이다.
현재 동기적으로 작성된 코드는 다음과 같다.
public Optional<User> getUserById(String id){
return userRepository.findById(id)
.map(user -> {
var image = imageRepository.findById(user.getProfileImageId())
.map(imageEntity -> {
return new Image(imageEntity.getId(), imageEntity.getName(), imageEntity.getUrl());
});
var articles = articleRepository.findAllByUserId(user.getId())
.stream().map(articleEntity ->
new Article(articleEntity.getId(), articleEntity.getTitle(), articleEntity.getContent())).toList();
var followCount = followRepository.countByUserID(user.getId());
return new User(
user.getId(),
user.getName(),
user.getAge(),
image,
articles,
followCount
);
});
}
Repository에서 가져올 때마다 1초 정도 시간이 걸린다고 생각하여 1초 정도 Thread.sleep을 사용했고, 조회한 값들을 다른 객체에 넣어서 반환하는 메서드이다.
차례대로 1초씩 3번 호출하기 때문에 적어도 3초 이상의 시간이 소모될 것이다.
void testGetUser(){
//given
String userId = "1234";
//when
Optional<User> optionalUser = userBlockingService.getUserById(userId);
//then
assertFalse(optionalUser.isEmpty());
var user = optionalUser.get();
assertEquals(user.getName(), "sk");
assertFalse(user.getProfileImage().isEmpty());
assertFalse(user.getProfileImage().isEmpty());
var image = user.getProfileImage().get();
assertEquals(image.getId(), "image#1000");
assertEquals(image.getName(), "profileImage");
assertEquals(image.getUrl(), "https://avatars.githubusercontent.com/u/98071131?s=400&u=9107a0b50b52da5bbc8528157eed1cca34feb3c5&v=4");
assertEquals(2, user.getArticleList().size());
assertEquals(1000, user.getFollowCount());
}
해당 테스트 코드의 시간을 확인 했을 때 4sec 74ms가 소모되었다.
이제 변경해보도록 하자.
일단 각 Repository를 CompletableFuture을 반환하도록 메서드를 변경했다.
@SneakyThrows
public CompletableFuture<Optional<UserEntity>> findById(String userId){
return CompletableFuture.supplyAsync(() -> {
log.info("UserRepository.findById: {}", userId);
try {
Thread.sleep(1000);
} catch (InterruptedException e) {
throw new RuntimeException(e);
}
var user = userMap.get(userId);
return Optional.ofNullable(user);
});
}
현재 에러만 잡아서 변경해놓은 코드는 다음과 같다.
@SneakyThrows
public Optional<User> getUserById(String id){
return userRepository.findById(id).get()
.map(this::getUser);
}
@SneakyThrows
private User getUser(UserEntity user){
var image = imageRepository.findById(user.getProfileImageId()).get()
.map(imageEntity -> {
return new Image(imageEntity.getId(), imageEntity.getName(), imageEntity.getUrl());
});
var articles = articleRepository.findAllByUserId(user.getId()).get()
.stream().map(articleEntity ->
new Article(articleEntity.getId(), articleEntity.getTitle(), articleEntity.getContent())).toList();
var followCount = followRepository.countByUserID(user.getId()).get();
return new User(
user.getId(),
user.getName(),
user.getAge(),
image,
articles,
followCount
);
}
이거를 Repository에 접근 할 때마다 CompletableFuture를 사용했다.
@SneakyThrows
public CompletableFuture<Optional<User>> getUserById(String id){
return userRepository.findById(id)
.thenCompose(this::getUser);
}
@SneakyThrows
private CompletableFuture<Optional<User>> getUser(Optional<UserEntity> userEntityOptional) {
if (userEntityOptional.isEmpty()) {
return CompletableFuture.completedFuture(Optional.empty());
}
var userEntity = userEntityOptional.get();
var imageFuture = imageRepository.findById(userEntity.getProfileImageId())
.thenApplyAsync(imageEntityOptional ->
imageEntityOptional.map(imageEntity ->
new Image(imageEntity.getId(), imageEntity.getName(), imageEntity.getUrl())
)
);
var articlesFuture = articleRepository.findAllByUserId(userEntity.getId())
.thenApplyAsync(articleEntities ->
articleEntities.stream()
.map(articleEntity ->
new Article(articleEntity.getId(), articleEntity.getTitle(), articleEntity.getContent())
)
.collect(Collectors.toList())
);
var followCountFuture = followRepository.countByUserID(userEntity.getId());
return CompletableFuture.allOf(imageFuture, articlesFuture, followCountFuture)
.thenAcceptAsync(v -> {
log.info("Three futures are completed");
})
.thenRunAsync(() -> {
log.info("Three futures are also completed");
})
.thenApplyAsync(v -> {
try {
var image = imageFuture.get();
var articles = articlesFuture.get();
var followCount = followCountFuture.get();
return Optional.of(
new User(
userEntity.getId(),
userEntity.getName(),
userEntity.getAge(),
image,
articles,
followCount
)
);
} catch (Exception e) {
throw new RuntimeException(e);
}
});
}
}
이렇게 변경하고 전에 사용했던 테스트 코드의 시간을 측정해보았다.
거의 반으로 줄어든 2sec 66ms가 나왔다.
전 코드에서 Repository에 접근 하는 것을 기다리기 보다 비동기적으로 실행한다면 시간을 크게 줄일 수 있는 것을 볼 수 있었던 것 같다.