search

jam2in / arcus-java-client

Arcus Java client
Apache License 2.0
0 stars 0 forks source link

Feature : Add unit tests for SMGet with ElementMultiFlagsFilter #37

Closed aiceru closed 7 years ago

aiceru commented 8 years ago

asyncBopSortMergeGet 에서 ElementMultiFlagsFilter 를 사용하는 경우에 대한 unit test 를 추가하였습니다. 더불어, 문서에서 ElementMultiFlagsFilter 를 사용할 수 있는 케이스에 asyncBopSortMergeGet 을 추가하였습니다.

jhpark816 commented 8 years ago

review 완료..

whchoi83 commented 7 years ago

리뷰 완료 했습니다

jhpark816 commented 7 years ago

@whchoi83 여기 PR은 review가 끝난 상태라 merge해도 될 것 같습니다. 단, CI 검사가 실패하는 이유가 무엇인지 알고 있나요 ?

whchoi83 commented 7 years ago

기존에 생성되었던 캐시 때문에 map 기능이 추가되지 않은 arcus-memcached 가 구동되어 그렇습니다. travis 에서 사용하는 쉘에 branch 가 다르면 기존 캐시를 삭제하고 다시 git clone 을 받도록 되어있는데 이 현상이 왜 발생하는지 잘 모르겠습니다.

CI가 실패하면 직접 들어가서 캐시를 삭제하고 다시 build 해서 동일하게 실패하는지 확인해봐야 합니다.

확인 결과 본 PR은 정상적으로 CI 통과됩니다. merge 하도록 하곘습니다