search

JuyeoungJun / cron-monitoring

for cron-monitoring
0 stars 0 forks source link

feat: 크론 로그 조회 기능 추가 - [merged] #105

Closed JuyeoungJun closed 3 years ago

JuyeoungJun commented 3 years ago

In GitLab by @gm2202985 on Jun 23, 2021, 11:27

Merges feat/54-cron-lookup -> feat/46-cron-process

Resolves: #54

Issue Number

54

What?

크론 로그 조회 기능 추가

Why?

크론 로그 조회 기능 구현

How?

Influx DB 를 통해 로그 조회

Testing?

JUnit Test를 통해 진행

JuyeoungJun commented 3 years ago

In GitLab by @gm2202983 on Jun 23, 2021, 11:40

Commented on src/main/java/gabia/cronMonitoring/repositoryImpl/CronLogRepositoryImpl.java line 49

조회하는 로그의 수가 적다면 상관없겠지만 많은 수의 로그를 갖고 오게 될거라 예상되기에 ArrayList 보다 LinkedList가 더 빠를 수 있다고 생각합니다!

JuyeoungJun commented 3 years ago

In GitLab by @gm2202983 on Jun 23, 2021, 11:41

approved this merge request

JuyeoungJun commented 3 years ago

In GitLab by @gm2202981 on Jun 23, 2021, 11:43

approved this merge request

JuyeoungJun commented 3 years ago

In GitLab by @gm2202985 on Jun 23, 2021, 12:04

Commented on src/main/java/gabia/cronMonitoring/repositoryImpl/CronLogRepositoryImpl.java line 49

changed this line in version 2 of the diff

JuyeoungJun commented 3 years ago

In GitLab by @gm2202985 on Jun 23, 2021, 12:04

added 3 commits

Compare with previous version

JuyeoungJun commented 3 years ago

In GitLab by @gm2202985 on Jun 23, 2021, 12:15

added 1 commit

Compare with previous version

JuyeoungJun commented 3 years ago

In GitLab by @gm2202985 on Jun 23, 2021, 12:30

added 1 commit

Compare with previous version

JuyeoungJun commented 3 years ago

In GitLab by @gm2202985 on Jun 23, 2021, 12:31

added 1 commit

Compare with previous version

JuyeoungJun commented 3 years ago

In GitLab by @gm2202985 on Jun 23, 2021, 13:03

added 1 commit

Compare with previous version

JuyeoungJun commented 3 years ago

In GitLab by @gm2202985 on Jun 23, 2021, 13:03

Commented on src/main/java/gabia/cronMonitoring/repositoryImpl/CronLogRepositoryImpl.java line 49

확인 했습니다.

JuyeoungJun commented 3 years ago

In GitLab by @gm2202985 on Jun 23, 2021, 13:06

mentioned in commit 30dfd52185d77b36296b53bc1c6b79e94957ebef