search

paust-team / paust-db

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

Change broadcast method of client from BroadcastTxSync to BroadcastTxCommit #112

Closed dragon0170 closed 5 years ago

dragon0170 commented 5 years ago

현재 Client interface를 구현한 httpclient.go에는 Put function에서 tx를 broadcast할 때 BroadcastTxSync 함수를 사용합니다. 이는 tendermint consensus의 CheckTx 과정을 거친 뒤 response를 return하는 함수입니다. 하지만 CheckTx에서는 데이터가 적절한 format인지만 확인을 하고 실제로 데이터가 put되는 것은 Commit이 일어났을 때 입니다. 따라서 Put fucntion에서는 BroadcastTxSync대신 BroadcastTxCommit 함수를 사용하는 것이 더 적절한 것으로 보입니다.

dragon0170 commented 5 years ago

BroadcastTxCommit으로 바뀐다면 데이터가 실제로 put이 되기까지의 시간을 측정하는 데 유용할 것 같습니다. 테스트 용도로 client CLI에서 put, query, fetch를 할 때 소요된 시간을 출력하면 좋을 것 같네요.