Skip to content

Commit ae8f472

Browse files
committed
refactor: Scheduler에 TRACE_ID가 기본으로 남도록 수정한다
1 parent 83321d0 commit ae8f472

4 files changed

Lines changed: 30 additions & 14 deletions

File tree

src/main/kotlin/org/gitanimals/rank/app/GivePointToGuildFacade.kt

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,13 @@
11
package org.gitanimals.rank.app
22

33
import org.gitanimals.core.IdGenerator
4-
import org.gitanimals.core.filter.MDCFilter.Companion.TRACE_ID
54
import org.gitanimals.rank.domain.GuildContributionRankService
65
import org.gitanimals.rank.domain.RankQueryRepository
76
import org.gitanimals.rank.domain.RankQueryRepository.RankType.WEEKLY_GUILD_CONTRIBUTIONS
87
import org.gitanimals.rank.domain.history.RankHistoryService
98
import org.gitanimals.rank.domain.history.request.InitRankHistoryRequest
109
import org.gitanimals.rank.domain.response.RankResponse
1110
import org.slf4j.LoggerFactory
12-
import org.slf4j.MDC
1311
import org.springframework.scheduling.annotation.Scheduled
1412
import org.springframework.stereotype.Component
1513

@@ -27,7 +25,6 @@ class GivePointToGuildFacade(
2725
@Scheduled(cron = "0 0 23 * * SUN")
2826
fun givePointsToGuildUsers() {
2927
runCatching {
30-
MDC.put(TRACE_ID, IdGenerator.generate().toString())
3128
val guildRankWithIds = rankQueryRepository.findAllRank(
3229
rankStartedAt = 0,
3330
limit = 2,
@@ -63,8 +60,6 @@ class GivePointToGuildFacade(
6360
rankQueryRepository.initialRank(WEEKLY_GUILD_CONTRIBUTIONS)
6461
}.onFailure {
6562
logger.info("[GivePointToGuildFacade] Fail to awarded point. ${it.message}", it)
66-
}.also {
67-
MDC.remove(TRACE_ID)
6863
}
6964
}
7065

src/main/kotlin/org/gitanimals/rank/app/GivePointToUsersFacade.kt

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,13 @@
11
package org.gitanimals.rank.app
22

33
import org.gitanimals.core.IdGenerator
4-
import org.gitanimals.core.filter.MDCFilter.Companion.TRACE_ID
54
import org.gitanimals.rank.domain.RankQueryRepository
65
import org.gitanimals.rank.domain.RankQueryRepository.RankType.WEEKLY_USER_CONTRIBUTIONS
76
import org.gitanimals.rank.domain.UserContributionRankService
87
import org.gitanimals.rank.domain.history.RankHistoryService
98
import org.gitanimals.rank.domain.history.request.InitRankHistoryRequest
109
import org.gitanimals.rank.domain.response.RankResponse
1110
import org.slf4j.LoggerFactory
12-
import org.slf4j.MDC
1311
import org.springframework.scheduling.annotation.Scheduled
1412
import org.springframework.stereotype.Component
1513

@@ -26,7 +24,6 @@ class GivePointToUsersFacade(
2624
@Scheduled(cron = "0 0 23 * * SUN")
2725
fun givePointToUsers() {
2826
runCatching {
29-
MDC.put(TRACE_ID, IdGenerator.generate().toString())
3027
val userRankWithIds = rankQueryRepository.findAllRank(
3128
rankStartedAt = 0,
3229
limit = 2,
@@ -55,7 +52,7 @@ class GivePointToUsersFacade(
5552
rank = it.rank,
5653
prize = getPoint(it),
5754
rankType = RankQueryRepository.RankType.WEEKLY_USER_CONTRIBUTIONS,
58-
winnerId = it.id.toLong(),
55+
winnerId = it.id.toLong(),
5956
winnerName = it.name,
6057
)
6158
}
@@ -64,8 +61,6 @@ class GivePointToUsersFacade(
6461
rankQueryRepository.initialRank(WEEKLY_USER_CONTRIBUTIONS)
6562
}.onFailure {
6663
logger.info("[GivePointToUsersFacade] Fail to awarded point. ${it.message}", it)
67-
}.also {
68-
MDC.remove(TRACE_ID)
6964
}
7065
}
7166

src/main/kotlin/org/gitanimals/render/infra/NewPetDropRateDistributionReport.kt

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package org.gitanimals.render.infra
22

3-
import org.gitanimals.core.IdGenerator
43
import org.gitanimals.core.filter.MDCFilter.Companion.TRACE_ID
54
import org.gitanimals.core.instant
65
import org.gitanimals.render.domain.PersonaStatisticService
@@ -49,8 +48,6 @@ class NewPetDropRateDistributionReport(
4948
}
5049

5150
private fun publishDropRateDistribution(days: Long, type: Type) {
52-
MDC.put(TRACE_ID, IdGenerator.generate().toString())
53-
5451
logger.info("[NewPetDropRateDistributionReport] Aggregate yesterday days pet distribution drop rate...")
5552

5653
val createdAt = instant().minus(days, ChronoUnit.DAYS).truncatedTo(ChronoUnit.DAYS)
Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
package org.gitanimals.supports.schedule
2+
3+
import org.aspectj.lang.ProceedingJoinPoint
4+
import org.aspectj.lang.annotation.Around
5+
import org.aspectj.lang.annotation.Aspect
6+
import org.gitanimals.core.IdGenerator
7+
import org.gitanimals.core.filter.MDCFilter.Companion.TRACE_ID
8+
import org.slf4j.MDC
9+
import org.springframework.core.annotation.Order
10+
import org.springframework.scheduling.annotation.Scheduled
11+
import org.springframework.stereotype.Component
12+
13+
@Aspect
14+
@Component
15+
@Order(value = Int.MIN_VALUE)
16+
class SchedulerTraceIdAspect {
17+
18+
@Around("@annotation(scheduled)")
19+
fun putMdcFilter(joinPoint: ProceedingJoinPoint, scheduled: Scheduled): Any? {
20+
return runCatching {
21+
MDC.put(TRACE_ID, IdGenerator.generate().toString())
22+
joinPoint.proceed()
23+
}.also {
24+
MDC.remove(TRACE_ID)
25+
}.onFailure {
26+
throw it
27+
}
28+
}
29+
}

0 commit comments

Comments
 (0)