You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@couchdb.apache.org by va...@apache.org on 2017/10/02 20:45:04 UTC

couchdb-khash git commit: Fix iterator expiry test [Forced Update!]

Repository: couchdb-khash
Updated Branches:
  refs/heads/fix-iterator-test 8048aaf74 -> 5ef2909c2 (forced update)


Fix iterator expiry test

This test started failing with Erlang 20.0 release. The reason is that opaaque
NIF resources stopped being identifed as empty binaries in Erlang so previously
it matched but once refs were used it stopped matching.

Fixes #855


Project: http://git-wip-us.apache.org/repos/asf/couchdb-khash/repo
Commit: http://git-wip-us.apache.org/repos/asf/couchdb-khash/commit/5ef2909c
Tree: http://git-wip-us.apache.org/repos/asf/couchdb-khash/tree/5ef2909c
Diff: http://git-wip-us.apache.org/repos/asf/couchdb-khash/diff/5ef2909c

Branch: refs/heads/fix-iterator-test
Commit: 5ef2909c294ba68940724175cda158a6b00d54d4
Parents: 0ca7f7e
Author: Nick Vatamaniuc <va...@apache.org>
Authored: Mon Oct 2 16:37:56 2017 -0400
Committer: Nick Vatamaniuc <va...@apache.org>
Committed: Mon Oct 2 16:44:59 2017 -0400

----------------------------------------------------------------------
 test/khash_test.erl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/couchdb-khash/blob/5ef2909c/test/khash_test.erl
----------------------------------------------------------------------
diff --git a/test/khash_test.erl b/test/khash_test.erl
index 20bd785..f19c858 100644
--- a/test/khash_test.erl
+++ b/test/khash_test.erl
@@ -240,7 +240,7 @@ no_expiration_iterators_test_() ->
                     }
                 end,
                 fun({ok, H, I, Err}) ->
-                    {ok, I} = khash:iter(H),
+                    {ok, _OtherI} = khash:iter(H),
                     {foo, bar} = khash:iter_next(I),
                     end_of_table = khash:iter_next(I),
                     {