[OAK-6851] Disabled index with reindex flag enabled should not lead to reindex report

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.7.10, 1.8.0
• Component/s: indexing
• Assignee: Chetan Mehrotra
• Reporter: Chetan Mehrotra
• Created: 20/Oct/17 11:43
• Updated: 27/Oct/17 10:06
• Resolved: 20/Oct/17 11:45

Description

Currently if any index is disabled but has reindex flag enabled then it leads to following log entry for each commit (assuming disabled index was sync)

20.10.2017 13:15:40.031 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing completed
20.10.2017 13:15:40.046 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Indexing report

20.10.2017 13:15:40.046 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing completed
20.10.2017 13:15:40.056 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Indexing report

20.10.2017 13:15:40.056 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing completed
20.10.2017 13:15:40.068 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Indexing report

20.10.2017 13:15:40.068 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing completed
20.10.2017 13:15:40.076 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Indexing report

20.10.2017 13:15:40.076 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing completed
20.10.2017 13:15:40.092 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Indexing report

20.10.2017 13:15:40.092 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing completed
20.10.2017 13:15:40.103 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Indexing report

20.10.2017 13:15:40.103 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing completed
20.10.2017 13:15:40.123 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Indexing report

20.10.2017 13:15:40.123 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing completed
20.10.2017 13:15:40.131 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Indexing report

20.10.2017 13:15:40.131 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing completed
20.10.2017 13:15:40.137 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Indexing report

20.10.2017 13:15:40.137 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing completed
20.10.2017 13:15:40.143 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Indexing report

20.10.2017 13:15:40.143 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing completed
20.10.2017 13:15:40.149 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Indexing report

20.10.2017 13:15:40.149 *INFO* [FelixStartLevel] org.apache.jackrabbit.oak.plugins.index.IndexUpdate Reindexing complet

The empty lines are due to report not logging details if reindex index entryCount is 0. As a fix reindex logic should totally ignore disabled indexes