Multiple query constraints
Reported by Ben Hood | November 17th, 2010 @ 03:42 PM | in 0.9.3
Ticket #2 introduced the notion of a QueryConstraint, whereby the API takes a sequence of constraints. However, working out the semantics of a multi-dimensional predicate effectively blew the scope of that bug, so I'm creating this ticket to cover that subject matter. Basically we need a sane interpretation of multiple query constraints.
Comments and changes to this ticket
-
Ben Hood January 17th, 2011 @ 12:01 PM
- Assigned user set to “Paul Jones”
-
-
Ben Hood January 17th, 2011 @ 01:14 PM
(from [4d3631857e5a4440d8c9f5dc1cfb82b390b04bb5]) [#148] Kernel changes necessary for multiple constraints. https://github.com/lshift/diffa/commit/4d3631857e5a4440d8c9f5dc1cfb...
-
Ben Hood January 17th, 2011 @ 03:48 PM
(from [ed70358516870231cc8e9475a83bde0edaf37ff2]) [#148] Fix components around the kernel to support multiple constraints. https://github.com/lshift/diffa/commit/ed70358516870231cc8e9475a83b...
-
Paul Jones January 17th, 2011 @ 03:50 PM
The basics of this work have been completed. I haven't added any test coverage for multiple constraints yet though - I want to layer this on top of the work done in #149, so I'm going to wait for that to land before proceeding.
-
-
Ben Hood January 18th, 2011 @ 10:03 AM
(from [7a5c5a1a7f6022572e37d38ed925cb1d144955c4]) [#148] Fix configuration problems in the demo participant now that category types are more important. https://github.com/lshift/diffa/commit/7a5c5a1a7f6022572e37d38ed925...
-
Ben Hood January 18th, 2011 @ 10:03 AM
(from [e3c5d1fa7dbad85ec408b5ad6c3a6e1fc22a4426]) [#148] Sync with master. Still need to fix the demo participant test. https://github.com/lshift/diffa/commit/e3c5d1fa7dbad85ec408b5ad6c3a...
-
Paul Jones January 18th, 2011 @ 10:07 AM
- State changed from “open” to “resolved”
- Assigned user changed from “Paul Jones” to “Ben Hood”
The latest master has been integrated into this bug, and given the high likelihood of bitrot for this bug, I think it may be best to defer any necessary testing improvements to another bug (or re-open this one after a courtesy merge).
-
Ben Hood January 18th, 2011 @ 04:12 PM
QA notes:
- CategoryFunctionRegistry
- Should the Integer category not be included here?
- JSONEncodingUtilsTest
- You've added {de}serializeWireAggregateRequest to the library, but there doesn't seem to be a test for it
- FileParticipant:
- Why are the assertions about constraints still in the file?
- General
- It might be a good idea to walk me through the Participant trait verbally
- If the scope of this ticket needs to be reduced, then we should
create follow on tickets before this is signed off. This covers:
- Tests for multiple category constraints
- Using JUnit theories
- Consistently eliminating SortedMaps (see #159)
- Style:
- AtrributeUtil, CategoryFunctionRegistry and WireAggregateRequest adon't have consistent license headers
- CategoryFunctionRegistry
-
Ben Hood January 18th, 2011 @ 04:15 PM
- State changed from “resolved” to “open”
- Assigned user changed from “Ben Hood” to “Paul Jones”
-
Ben Hood January 18th, 2011 @ 05:39 PM
- Milestone cleared.
- Milestone order changed from “28” to “0”
-
Ben Hood January 19th, 2011 @ 09:52 AM
In order to keep this ticket tractable, we'll merge the current state into master and follow up this ticket with #158 in order to introduce tests for multiple categories using a (to be introduced) theory framework.
-
Ben Hood January 19th, 2011 @ 12:26 PM
(from [34292714f62dc91f681942acdc9fb65564a39f7f]) [#148] Implement QA remarks: - Add integer categories to CategoryFunctionRegistry - Add test for new method in JSONEncodingUtils - Clarify constraints in FileParticipant - Add license headers https://github.com/lshift/diffa/commit/34292714f62dc91f681942acdc9f...
-
Paul Jones January 19th, 2011 @ 12:27 PM
- State changed from “open” to “resolved”
- Assigned user changed from “Paul Jones” to “Ben Hood”
QA remarks implemented.
-
-
-
Ben Hood February 4th, 2011 @ 02:48 PM
- Milestone set to “0.9.3”
- Milestone order changed from “13” to “0”
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
A Real Time Differencing Tool.
People watching this ticket
Ben Hood
Referenced by
-
2
Generalize entity partition criteria and granularity function
#148 - to remove the need for limiting
constraints to jus...
-
148
Multiple query constraints
(from [4d3631857e5a4440d8c9f5dc1cfb82b390b04bb5])
[#148] ...
-
148
Multiple query constraints
(from [ed70358516870231cc8e9475a83bde0edaf37ff2])
[#148] ...
-
148
Multiple query constraints
(from [7a5c5a1a7f6022572e37d38ed925cb1d144955c4])
[#148] ...
-
148
Multiple query constraints
(from [e3c5d1fa7dbad85ec408b5ad6c3a6e1fc22a4426])
[#148] ...
-
158
Data Driven Policy Tests
Following on from #148 and #154, we need to:
-
148
Multiple query constraints
(from [34292714f62dc91f681942acdc9fb65564a39f7f])
[#148] ...
-
148
Multiple query constraints
(from [96fc40acaf55d772c45ae4e12b60705e421e32e6])
Merged ...
-
143
AMQP Transport
Also, don't forget to merge master back into your branch ...
-
143
AMQP Transport
Updated for QA notes, merged changes for #148 from
master