[LUCENE-9513] Use seconds instead of millisecs

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 9.0, 8.6.2
• Fix Version/s: 9.0, 8.7
• Component/s: core/other
• Lucene Fields: New, Patch Available
• Assignee: Unassigned
• Reporter: Frank Zhu
• Created: 09/Sep/20 07:11
• Updated: 28/Aug/22 16:07
• Resolved: 09/Sep/20 14:40

Description

In RangeFacetsExample, there is a trivial mistake where

private final long nowSec is supposed to be secs.

But it uses as millisecs.

It doesn't affect the output due to the scale change (It will just make PAST_HOUR, PAST_SIX_HOURS not as intended). Still it's better to be correct.

 

I've created a pull request on github with the simple fix.

https://github.com/apache/lucene-solr/pull/1846

Issue Links

• links to: GitHub Pull Request #1846