Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-10538

TopN is not being used in getTopChildren()

Details

    • Bug
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • None
    • None
    • modules/facet
    • None
    • New

    Description

      When looking at the overridden implementation getTopChildren(int topN, String dim, String... path) in RangeFacetCounts, I found that the topN parameter is not being used in the code, and the unit tests did not test this function properly. I will create a PR to fix this, and will look into other overridden implementations and see if they have the same issue. Please let me know if there is any question. Thanks!

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              yutinggan Yuting Gan
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 2h
                  2h