Permalink
Browse files

ui: respect limit field

Can't look up traces by span.

respects the "limit" field.

changed the spec to expect proper "limit" field. I don't know why it
didn't in the first place. Seems like the test was incorrect too.

Author: @mosesn
Fixes #176
URL: #176
  • Loading branch information...
mosesn authored and Franklin Hu committed Oct 15, 2012
1 parent b49ec2d commit 954c8acf3d07003efaa1473c9126a6498f37bae0
Showing with 4 additions and 1 deletion.
  1. +4 −1 zipkin-query-core/src/main/scala/com/twitter/zipkin/query/QueryService.scala
@@ -125,7 +125,10 @@ class QueryService(storage: Storage, index: Index, aggregates: Aggregates, adjus
}
case head :: Nil => {
/* One query: just run it */
- head.execute(index).map {
+ (head match {
+ case s: SpanSliceQuery => s.copy(limit = limit)
+ case a: AnnotationSliceQuery => a.copy(limit = limit)
+ }).execute(index).map {
constructQueryResponse(_, limit, order)
}.flatten
}

0 comments on commit 954c8ac

Please sign in to comment.