Conversation
…tering 쿼리를 추가 issue : #272 기능 추가 : CustomizedMetaDataRepository 내 findByProjectIdWithPagingAndFiltering API를 추가하여, 파라미터 값으로 받은 page, size, projectId 를 통해 해당 프로젝트 아이디를 가진 메타데이터 중 page 값에 해당 하는 메타데이터들을 size 만큼 받아 온다. MeataData QA로 실제 DB로 접근하여 테스팅시, * 기존 방식으로 10만개 기준 2sec 380ms가 소요되지만 size를 20으로 규정 후 paging 기법으로 찾으니, 330ms만에 해당 페이징 내에 있는 메타데이터 리스트를 찾아올 수 있었다. 추가 방향 : 1. 필터링 값들을 동적으로 적용할 수 있도록 수정한다. 2. 개선율을 100만개 단위로 계산해 보고, 성능향상을 기록한다.
BEOKS
requested changes
May 14, 2022
server/DSMP/src/main/java/com/knuipalab/dsmp/metadata/CustomizedMetaDataRepositoryImpl.java
Show resolved
Hide resolved
server/DSMP/src/main/java/com/knuipalab/dsmp/metadata/CustomizedMetaDataRepositoryImpl.java
Outdated
Show resolved
Hide resolved
server/DSMP/src/main/java/com/knuipalab/dsmp/metadata/CustomizedMetaDataRepositoryImpl.java
Show resolved
Hide resolved
issue : #273 Controller 영역에서 Request Parm 통한 page&size 동적으로 처리 가능하도록 수정
머신러닝 API 테스트 코드의 경우 빈 생성에 필요한 내부 클래스 인스턴스인 AsyncMetaDataSampler가 빈으로 생성되어 있지 않아 이를 주입하였다.
BEOKS
requested changes
May 18, 2022
server/DSMP/src/main/java/com/knuipalab/dsmp/metadata/MetaDataApiController.java
Outdated
Show resolved
Hide resolved
issue : #272 오류 수정 내역 : 1. 현재 기존에 만들었던 함수 findByProjectIdWithPagingAndFiltering에서 현재 필터링 기능이 없어 네이밍을 findByProjectIdWithPaging으로 수정하였음 2. 기존의 @GetMapping("api/MetaData/{projectId}")으로 진행했던 testcode에서 findByProjectId 기능을 findByProjectIdWithPaging 기능으로 수정하여 진행하였음 (컨트롤러&서비스 단). => projectId와 페이징 정보를 기반으로 메타데이터 리스트를 반환한다.
BEOKS
requested changes
May 18, 2022
...DSMP/src/test/java/com/knuipalab/dsmp/controller/metadata/api/MetaDataApiControllerTest.java
Outdated
Show resolved
Hide resolved
...DSMP/src/test/java/com/knuipalab/dsmp/controller/metadata/api/MetaDataApiControllerTest.java
Outdated
Show resolved
Hide resolved
server/DSMP/src/test/java/com/knuipalab/dsmp/service/metadata/MetaDataServiceTest.java
Show resolved
Hide resolved
LDEV4966
changed the title
issue : #273 문제해결 점 : 기존에 제거한 findByProjectId 기능을 원상복구 하고, uri 중복을 피하기 위해 pagination기능의 uri를 api/MetaData/{projectId}/pagination로 수정
LDEV4966
commented
May 19, 2022
...DSMP/src/test/java/com/knuipalab/dsmp/controller/metadata/api/MetaDataApiControllerTest.java
Outdated
Show resolved
Hide resolved
BEOKS
approved these changes
May 20, 2022
BEOKS
pushed a commit
that referenced
this pull request
May 28, 2022
issue : #273 Controller 영역에서 Request Parm 통한 page&size 동적으로 처리 가능하도록 수정
페이지 요청 정보를 레퍼 클래스로 만들고 메타데이터 리스트를 업로드 할 때, 함수 분할과 람다를 이용하여 가독성 향상
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
issue : #272
기능 추가 :
CustomizedMetaDataRepository 내 findByProjectIdWithPagingAndFiltering API를 추가하여, 파라미터 값으로 받은 page, size, projectId 를 통해 해당 프로젝트 아이디를 가진 메타데이터 중 page 값에 해당 하는 메타데이터들을 size 만큼 받아 온다.
MeataData QA로 실제 DB로 접근하여 테스팅시,
추가 방향 :