forked from rackerlabs/repose
-
Notifications
You must be signed in to change notification settings - Fork 2
/
technical.todo
38 lines (29 loc) · 1.85 KB
/
technical.todo
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
- Replace StringUtils with apache commons lang
-- Evaluate any of the other utils to see if they can also be replaced
- remove own timeunit enum
- replace object serializer with commons serialization utils
- repose-aggregator/commons/utilities
-- Make spring dependencies optional. The user of utilities will need to explicitly
call out the spring dependencies.
-- Break the dependency on httpclient-connection-pool
- repose-aggregator/commons/classloader
-- Break the dependency on external
- Evaluate usage of code - ReposeLocalCacheMBean, ReposeLocalCache
[ ] Datastore Implementation Improvements
[ ] Improve error and null checks in datastore implementations
[ ] Change Datastore exceptions to non-runtime exceptions
[ ] PUT behavior allows empty strings returns a 202, but a GET will return a 404. Is this correct?
[ ] DELETE of a non-existent item currently returns a 202, what should it really do??? (Not a 202 IMHO)
[ ] Different behavior of DD Filter vs DD Service (DDService really should NOT be returning 500 on bad request data)
[ ] GET w/ bad data (no key, no X-PP-Host-Key header, invalid key) returns 400 in filter, 500 w/ msg in service
[ ] PUT w/ bad data (no key, no X-PP-Host-Key header, invalid key, large body) returns 400 in filter, 500 w/ msg in service
[ ] Additional scenarios
[ ] PUT with no TTL, what is our behavior? Add a functional test.
[ ] Pull request feedback of DD modularization story
[ ] ClusterView - modify members() to return a List vs an array
[ ] HashRingDatastore - appending original exception message, and also wrapping it
[ ] ThreadSafeClusterView - evalute whether to remove the reference copy
- TEST IMPROVEMENTS
- ContentNormalization filter - 0% functional test coverage
[ ] DistDatastoreServiceTest
[ ] fix hardcoded datastore service endpoint