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

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2019/06/20 22:56:54 UTC

[GitHub] [accumulo] ctubbsii edited a comment on issue #1223: Monitor log recovery says invalid date

ctubbsii edited a comment on issue #1223: Monitor log recovery says invalid date
URL: https://github.com/apache/accumulo/issues/1223#issuecomment-504228075
 
 
   After a few minutes of my own investigation, I noticed that the original fix in #1028  did not fix the problem. It mistakenly changed the variable from `time` to `runtime` (presumably on the belief that the underlying POJO being presented in the REST interface was the class `RecoveryStatusInformation`, but it was actually `TabletServerRecoveryInformation`).
   
   The actual error was that the `time` field was being interpreted as an absolute time to be used with `new Date(time)`, when the `time` field was actually a time duration, and needed to be displayed as such.
   
   I have a fix for this, and will push it shortly.

----------------------------------------------------------------
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


With regards,
Apache Git Services