search

paust-team / paust-db

GNU General Public License v3.0
6 stars 5 forks source link

Decide whether to include the interfaces for testing in the DB interfaces #39

Closed kwjooo closed 5 years ago

kwjooo commented 5 years ago

Reference

19

개요 현재 paust db에서 Read와 Write시 각각 Iterator와 Batch를 사용하고 있으며 단일 Read와 Write에 대한 기능은 Test 상황에서는 필요하나 실제 어플리케이션 로직에서는 필요하지 않는 상황입니다. 이러한 상황에서 DB Interface에 단일 Read와 Write를 포함시키는 게 좋을 지에 대한 논의가 필요합니다.

1dennispark commented 5 years ago

저 같은 경우는 테스트에 필요한데 실제 상황에서 필요가 없다고 한다면 그냥 주석을 통해 For Test를 명시합니다. 이걸통해서 production이나 runtime 환경에서는 최대한 사용을 지양하도록 유도할 수 있다고 생각해요.

https://github.com/facebook/folly/blob/0c1847c1c7b9cb1160ec9efb2b0b59b24d5f8faf/folly/json.h#L190-L195

위의 경우와 같이 Printer For GTest 처럼 남겨놓는 케이스도 많습니다.

code-to-gold commented 5 years ago

제목 수정하였습니다.