)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"a1230d28f979154f7efc9412d2da490dd43dff49","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add \u0027limit_terms\u0027 to facets"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds \u0027limit_terms\u0027 to restrict options from elasticsearch. Default is"},{"line_number":10,"context_line":"30."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I09605924b6763f600fb335cf77ac7c0f86fef7d8"},{"line_number":13,"context_line":"Closes-Bug: #1501539"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a1a9d01_18ed6f8e","line":10,"updated":"2015-10-01 19:11:06.000000000","message":"Missed the default here is 0.","commit_id":"16cdc2b4f2c7c73ddbc0474da7e2e2ab68a3741b"}],"doc/source/searchlightapi.rst":[{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"7a61fd828dae540e5b7c1f80af997c603183b3be","unresolved":false,"context_lines":[{"line_number":522,"context_line":""},{"line_number":523,"context_line":"There is a limit on the number of ``options`` returned for fields that support"},{"line_number":524,"context_line":"facet terms. The default is 30, but is controlled by the ``limit_terms`` parameter."},{"line_number":525,"context_line":"A value of ``0`` means \u0027no limit\u0027."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9a1a9d01_9d6731ea","line":525,"updated":"2015-10-01 18:32:04.000000000","message":"See my other comment about default of 30.  Sorry I suggested that.","commit_id":"2cbe2ad9cdbf13a9ff7aa5b6b02f8477da4972d7"}],"searchlight/api/v1/search.py":[{"author":{"_account_id":7665,"name":"Travis Tripp","email":"os.travis.tripp@gmail.com","username":"travis.tripp"},"change_message_id":"7a61fd828dae540e5b7c1f80af997c603183b3be","unresolved":false,"context_lines":[{"line_number":119,"context_line":"            LOG.error(encodeutils.exception_to_unicode(e))"},{"line_number":120,"context_line":"            raise webob.exc.HTTPInternalServerError()"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    def facets(self, req, index_name\u003dNone, doc_type\u003dNone, limit_terms\u003d30):"},{"line_number":123,"context_line":"        try:"},{"line_number":124,"context_line":"            search_repo \u003d self.gateway.get_catalog_search_repo(req.context)"},{"line_number":125,"context_line":"            result \u003d search_repo.facets(index_name, doc_type, limit_terms)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_7d301de9","line":122,"updated":"2015-10-01 18:32:04.000000000","message":"After reading this: \n\nhttps://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations-bucket-terms-aggregation.html#_size\n\nI think we should stay with default of 0.  This gives some control, but I was hoping that we could say \"if more than X\" don\u0027t return any.  And we\u0027d just make it a freeform text entry.","commit_id":"2cbe2ad9cdbf13a9ff7aa5b6b02f8477da4972d7"}]}