OneBusAway/onebusaway-application-modules — pull request #2 by jeffmaki (2012-03-07 20:07:14+00:00)
Title: Onebusaway nyc app mods merge

Okay, for real this time--patches are in the referenced commits. Let's discuss how to handle version numbering, and also whether the few hacks I had to make to get GWT working are needed; maybe you've already fixed? Might also need to do a release on the GTFS and CSV modules--not 100% sure it was required, but seemed like it was--they're in our own private mvn repo now, but should be public.

But otherwise, should be a set of fairly clean, straightforward changes that shouldn't change anything for anybody else--or that's the hope at least... let's discuss...

Just like before, pasting in the original list E-mail below:

Hello OBA-ers,

Brian--this is mostly directed to you: I'm writing to submit a series
of smallish patches we made as part of OBA's integration with MTA Bus
Time to OBA trunk, hopefully to eventually merge these two codebases
back together. Patch is attached. I'm also happy to commit these if I
receive the go-ahead, otherwise feel free to merge them yourself.

Rough summary of patch contents are:

ServiceAlertsServiceImplTest: unit test for service alert changes

RoutesBeanServiceImpl: change to eliminate internal cached data
structures in favor of a "cacheable" method that does the same;
queryBean configurable minimum lucene score to keep

VehicleStatusBeanServiceImpl: informative debugging message only

RefreshableResources: new refreshable resource key for
WhereGeospatialServiceImpl

SiriService: change to enable unit testing

VehicleStatusServiceImpl: change to reset block state in
BlockVehicleLocationService if a record is received by the
VehicleStatusServiceImpl that has no trip/block assigned to it (e.g.
inference drops out) for the same vehicle ID

WhereGeospatialServiceImpl: made "refreshable", plus informative debug message

ServiceAlertsServiceImpl: switch to turn bundle persistence of service
alerts on/off; new time parameter to getServiceAlertIdsAsObject

BlockStatusServiceImpl: enlargement of search window for block list
that backs getBlocksForRoute(), to better handle vehicles assigned to
"late" trips (that weren't within the previously super tight bound
(instant?) specified as "time").

BlockGeospatialServiceImpl: change made to, when "refreshed", clear
the blockSequenceIndices so the old ones are not left in the internal
cache state.

SearchQueryBean: moved minimum lucene score to keep to this bean so
that it is configurable per query by the developer.