CategoryDescriptors should be strongly typed
Reported by Ben Hood | April 11th, 2011 @ 08:51 AM | in 0.9.3
Currently, bounded range CategoryDescriptors initialize themselves to be a typeless RangeCategoryConstraint. This causes an issue when a default bound is set in the config, e.g.:
<range-category name="bizDate" data-type="date" lower="2011-01-01" upper="2011-01-31"/>
This results in the following type error:
15:33:41.940 [akka:event-driven:dispatcher:global-10] ERROR PairActor:82 - FAILED to synchronise pair Staging7
scala.MatchError: RangeQueryConstraint(bizDate,2011-01-01,2011-01-31)
at net.lshift.diffa.kernel.indexing.LuceneVersionCorrelationStore$$anonfun$applyConstraints$1.apply(LuceneVersionCorrelationStore.scala:112) ~[diffa-kernel-0.9.3-SNAPSHOT.jar:na]
What should probably happen here is that type information in the XML is used to create a more appropriately typed RangeQueryConstraint.
Comments and changes to this ticket
-
Ben Hood April 11th, 2011 @ 10:27 AM
(from [9493988d4fb679e7f73c099813afa996cf6365f5]) [#202]: RangeCategoryDescriptors are typed when they are read out of config https://github.com/lshift/diffa/commit/9493988d4fb679e7f73c099813af...
-
Ben Hood April 11th, 2011 @ 11:08 AM
- State changed from “open” to “resolved”
- Assigned user changed from “Ben Hood” to “Paul Jones”
-
-
-
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
-
202
CategoryDescriptors should be strongly typed
(from [9493988d4fb679e7f73c099813afa996cf6365f5])
[#202]:...
-
202
CategoryDescriptors should be strongly typed
(from [2fb0bdc040cfd7ce5cca10c7857af6dfea248532])
Merge [...
-
202
CategoryDescriptors should be strongly typed
(from [8fa0f100421036805f3263649bbdfb6fcac984cd])
Merge m...