nfsantos commented on code in PR #2183:
URL: https://github.com/apache/jackrabbit-oak/pull/2183#discussion_r2038863715
##########
oak-search-elastic/src/main/java/org/apache/jackrabbit/oak/plugins/index/elastic/index/ElasticBulkProcessorHandler.java:
##########
@@ -268,113 +484,75 @@ public void beforeBulk(long executionId, BulkRequest
request, List<String> conte
}
@Override
- public void afterBulk(long executionId, BulkRequest request,
List<String> contexts, BulkResponse response) {
+ public void afterBulk(long executionId, BulkRequest request,
List<OperationContext> contexts, BulkResponse response) {
try {
- LOG.debug("Bulk with id {} processed in {} ms", executionId,
response.took());
+ LOG.debug("Bulk with id {} processed in {} ms", executionId,
response.took() / 1_000_000);
Review Comment:
Fixed, removed the division by 1000000. But `response.took()` seems to be
returning bogus values, in my tests, I was always seeing the same arbitrary
value. This value is received from the server, so most likely the problem is
not on the client side.
--
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.
To unsubscribe, e-mail: oak-dev-unsubscr...@jackrabbit.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
