Closed anirudh-raja closed 6 years ago
@BrianNichols Please can you check this out.
Your pull request has been accepted with modifications.
1) namespaces command line argument implies namespaces will be used for all transactions. Renamed namespaces to batchNamespaces to reflect that they will only be used for batch.
2) batch size (batch size * batchNamespaces length) no longer reflected command line batch size. Restored batch size to be command line batch size and choose each batch key namespace in round-robin fashion.
Note that all namespaces should be initialized with records when using batchNamespaces.
This will appear in the next java client release.
This was included in java client 4.1.7 release.
The benchmarking tool doesn't have support for multiple namespaces currently. We at Flipkart were benchmarking the performance of reading from multiple namespaces in a batch call to understand the performance of the same. Have added code to support multiple namespaces as a parameter.