search

jam2in / arcus-java-client

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

Fix timeout tests #8

Closed aiceru closed 8 years ago

aiceru commented 8 years ago
  1. Create workload generator class in BaseIntegrationTest.
  2. Fix each timeout tests to use workload generator so that the server becomes bottleneck.
  3. Change intended timeout duration to 1 ns instead of 1 ms to ensure timeout occurs.

Fix #5

Review

whchoi83 commented 8 years ago

리뷰가 늦어져서 죄송합니다. 깜박하고 있었네요. 😓 리뷰 완료했습니다.

한 가지 (소소한)의견으로는 WorkloadGenerator의 생성자에서 증가분을 인자로 받을 수 있게 하면 좋을 것 같습니다. 

public WorkloadGenerator(ArcusClient mc, int keySize, int inc, CountDownLatch latch) {

테스트 마다 부하를 덜 주거나 더 줄 필요가 있는 것들도 있지 않을까 하는 생각이 들어서 그렇습니다. 나중에 테스트가 추가되는 경우도 대비를 해서요. 당장은 크게 필요하지 않을 수도 있으니, 필요 하지 않다고 판단하시면 적용하지 않아도 괜찮을 것 같습니다.

aiceru commented 8 years ago

위 의견 반영하여 수정하였습니다~ 직관적인 사용을 위하여 loop count 의 의미로 추가하였습니다.

public WorkloadGenerator(ArcusClient mc, int size, int loop, CountDownLatch latch)
jhpark816 commented 8 years ago

@aiceru Travis CI build test가 ZK 문제로 완료되지 않는 것 같습니다. 한번 확인 부탁합니다.

aiceru commented 8 years ago

일단 merge 보류합니다. build fail 이 또 발생해서 계속 실행해보고 있는데, 결과가 50:50 정도로 일관성이 전혀 없어 보입니다. (거의 운빨인듯;;;;) 뭔가 규칙성이 있는지 좀 더 수행해 보고 노티하겠습니다.