You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@phoenix.apache.org by GitBox <gi...@apache.org> on 2021/05/06 00:52:20 UTC

[GitHub] [phoenix] jpisaac commented on a change in pull request #1211: PHOENIX-6432 Add support for additional load generators

jpisaac commented on a change in pull request #1211:
URL: https://github.com/apache/phoenix/pull/1211#discussion_r627007841



##########
File path: phoenix-pherf/src/main/java/org/apache/phoenix/pherf/workload/mt/operations/TenantOperationFactory.java
##########
@@ -164,7 +175,7 @@ public void funnel(TenantView tenantView, PrimitiveSink into) {
         }
 
         // This holds the info whether the tenant view was created (initialized) or not.
-        return BloomFilter.create(tenantViewFunnel, numTenants, 0.01);
+        return BloomFilter.create(tenantViewFunnel, numTenants, 0.0000001);

Review comment:
       https://hur.st/bloomfilter/?n=1000000&p=1.0E-7&m=&k=
   This shows that the space  requirements are not that large for 1M items, for that false-positive probability




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org