[t:4286], add comment

git-svn-id: file:///svn/toku/tokudb@37944 c7de825b-a66e-492c-adef-691d508d4ae1

newbrt/cachetable.c

@@ -1712,6 +1712,12 @@ static int cachetable_put_internal(
         for (p=ct->table[fullhash&(cachefile->cachetable->table_size-1)]; p; p=p->hash_chain) {
             count++;
             if (p->key.b==key.b && p->cachefile==cachefile) {
+                // Ideally, we would like to just assert(FALSE) here
+		// and not return an error, but as of Dr. Noga,
+		// cachetable-test2 depends on this behavior.
+		// To replace the following with an assert(FALSE)
+		// we need to change the behavior of cachetable-test2
+		//
                 // Semantically, these two asserts are not strictly right.  After all, when are two functions eq?
                 // In practice, the functions better be the same.
                 assert(p->flush_callback==flush_callback);