openspending / babbage

95%

master: 94%

LAST BUILD ON BRANCH bug/1300-dont-stream-csv-responses

branch: bug/1300-dont-stream-csv-responses

CHANGE BRANCH

x

Reset

• bug/1300-dont-stream-csv-responses
• 0.2.1
• 0.2.2
• 0.2.3
• 0.3.0
• 0.3.1
• 0.3.2
• 0.3.3
• 0.3.4
• 0.3.5
• 0.3.6
• bug/1299-aggregates-summaries-are-incorrect
• bug/1308-fix-routes-without-trailing-slash
• feature/1123-download-csv-from-any-visualization
• feature/1123-fix-bug-in-csv-downloads
• feature/add-types-support
• feature/better-join-syntax
• feature/optimize-member-queries
• fix-graco-exception
• fix/restore-summary-row
• maintenance
• master

Build # 125

Build Type

push

travis-ci

Committed by vitorbaptista

Commit Message

[#1300] Don't stream aggregate's CSV responses

Although it would be very important to stream the response, as CSVs can get big,
we can't do so with our current caching infrastructure that relies on pickling
the responses. Streaming responses can't be pickled.

I've done this change in a way that's trivial to come back to streaming,
whenever our caching infrastructure supports it.

openspending/openspending#1300

Run Details

5 of 5 new or added lines in 1 file covered. (100.0%)

731 of 773 relevant lines covered (94.57%)

1.89 hits per line