Not obvious that you need echoParams=all!
GoogleCodeExporter opened this issue · comments
What steps will reproduce the problem?
1.Use an external file that doesn't have echoParams=all
2.Do a query, only the first one is run, and no results.
3.If you look at solrmeter.log you will see the error.
What is the expected output? What do you see instead?
If I hack my external queries file to have echoParams=all, then it works.
I think you need better error handling around line
http://code.google.com/p/solrmeter/source/browse/trunk/sources/solrmeter/src/mai
n/java/com/plugtree/solrmeter/model/statistic/QueryLogStatistic.java#103 which
is where I am seeing the error.
Original issue reported on code.google.com by ep...@opensourceconnections.com
on 3 Aug 2011 at 9:50
Original comment by tflo...@gmail.com
on 4 Aug 2011 at 9:38
- Changed state: Accepted
- Added labels: Priority-High
- Removed labels: Priority-Medium
This issue was closed by revision r251.
Original comment by juan.gra...@gmail.com
on 5 Aug 2011 at 3:04
- Changed state: Fixed
Original comment by juan.gra...@gmail.com
on 5 Aug 2011 at 3:10
- Added labels: Milestone-Release-0.2.1
So, looking at r251, it doesn't seem like it actually adds the echoParams if it
is missing. You added to the example of course, but should SolrMeter add it if
I haven't???/
Original comment by ep...@opensourceconnections.com
on 5 Aug 2011 at 3:10
Hi,
In my opinion, echoParams=all shouldn't be forced (at least by default).
Currently the only drawback of using echoParams=none is that the "q" and "fq"
columns of the "Query Statistics" console will be empty. If you want to show
all the data, then you'll have to edit your queries file or change your request
handler configuration. Anyway, I'll create an issue to add a "Force
echoParams=all" switch to the configuration dialog.
Original comment by juan.gra...@gmail.com
on 5 Aug 2011 at 3:36
Original comment by juan.gra...@gmail.com
on 5 Aug 2011 at 3:41
- Changed state: Resolved
cool. One of the things I like about SolrMeter is it makes it easy for
novices to Solr to do some pretty sophisticated stuff. So I vote for sane
defaults that make it easy to use!
Original comment by ep...@opensourceconnections.com
on 5 Aug 2011 at 4:40
Original comment by tflo...@gmail.com
on 9 Oct 2012 at 10:48
- Added labels: Milestone-Release-0.3.0
- Removed labels: Milestone-Release-0.2.1