bug fix: checking partition id in Cluster.java #84

wants to merge 1 commit into


None yet
3 participants

shingon commented Jul 15, 2012

"numberOfTags" is always zero in getNumberOfTags(). So, this for loop statement was uselessness code. This patch is fix it.

icefury71 was assigned Jul 24, 2012


icefury71 commented Sep 25, 2012

Have you run all the Junit tests for this change ?

shingon commented Sep 26, 2012

Yes. I ran junit test (ver.0.96).
However, MySQL and Hadoop tests are error, because these softwares are not installed in my system.
Others are all success.

Error tests:

  • voldemort.store.mysql.MysqlStorageEngineTest
  • voldemort.store.readonly.mr.HadoopStoreBuilderCollisionTest
  • voldemort.store.readonly.mr.HadoopStoreBuilderTest

singhsiddharth commented Feb 12, 2014

This was fixed as part of this commit : f69def2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment