arcus-java-client icon indicating copy to clipboard operation
arcus-java-client copied to clipboard

ENHANCE: Change asyncSetPipedExist method logic.

Open brido4125 opened this issue 2 years ago • 6 comments

관련 이슈

  • jam2in/arcus-works#415

Motivation

기존 asyncSetPipedExist는 하나의 Operation 인스턴스만 지원한다. 이를 다른 Pipe 연산과 동일하게 Elem의 개수가 500개가 넘어가도 정상적인 api 처리를 해주는 로직으로 변경한다.

변경 지점

기존에 존재하는 asyncSetPipedExist의 연산 로직을 변경하였습니다.

우선 500개의 요소 단위로 하나의 연산을 구성하여 multi operation 인스턴스에 대한 지원이 가능하도록 변경하였습니다.

기존 로직과 달리 여러개의 op연산이 발생하기 때문에 다른 piped api들과 마찬가지로 mergedOperationStatus을 사용합니다.

brido4125 avatar Jun 22 '23 05:06 brido4125

@uhm0311 리뷰 바랍니다.

jhpark816 avatar Jul 24 '23 12:07 jhpark816

@uhm0311 리뷰 부탁드립니다.

brido4125 avatar Aug 03 '23 02:08 brido4125

@jhpark816 최종 리뷰 부탁드립니다.

brido4125 avatar Aug 09 '23 01:08 brido4125

@namsic asyncSetPipedExist()에서 future에 어떤 결과를 담아두어야 하는 지를 리뷰해 주세요.

jhpark816 avatar Aug 16 '23 01:08 jhpark816

@uhm0311 @namsic 리뷰 바랍니다.

jhpark816 avatar Aug 22 '23 11:08 jhpark816

@brido4125 rebase 한번 해 주세요.

jhpark816 avatar Mar 18 '24 10:03 jhpark816