keith-ratcliffe opened a new issue, #3630:
URL: https://github.com/apache/accumulo/issues/3630

   **Describe the bug**
   
   _ClientContext.getBatchWriterConfig(props)_ is incorrectly handling 
[BATCH_WRITER_LATENCY_MAX 
](https://github.com/apache/accumulo/blob/2.1/core/src/main/java/org/apache/accumulo/core/clientImpl/ClientContext.java#L370)
 and 
[BATCH_WRITER_TIMEOUT_MAX](https://github.com/apache/accumulo/blob/2.1/core/src/main/java/org/apache/accumulo/core/clientImpl/ClientContext.java#L370),
 such that the configured values are being converted from seconds to 
milliseconds twice, causing the stored values to be off by a factor of 1000
   
   **Versions (OS, Maven, Java, and others, as appropriate):**
    - Accumulo version: 2.1
   
   **To Reproduce**
   Use _ClientContext.getBatchWriterConfig(props)_ to build a BatchWriterConfig 
with the following properties
   ```
   batch.writer.timeout.max=300s
   batch.writer.latency.max=100s
   ```
   And observe that the resulting BWC's values are incorrect:
   ```
   [maxMemory=52428800, maxLatency=100000000, maxWriteThreads=3, 
timeout=300000000, durability=DEFAULT]
   ```
   
   **Expected behavior**
   
   The stored values for the BWC instance should be:
   - maxLatency = 100000
   - timeout =  300000
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to