sampottinger / co_opencampaigndata

API service for serving Colorado TRACER data for opencampaigndata.org
GNU General Public License v3.0
10 stars 1 forks source link

Transform datetimes while streaming instead of toArray #42

Closed sampottinger closed 11 years ago

sampottinger commented 11 years ago

As stated in line 216 of tracer_db_facade.js (commit b77b0cd523d9013dc9a035448c70325ba6db6410):

TODO(samnsparky): Stream results while doing transform

The create of Date objects should happen in stream instead of after a call to toArray. This has not had a noticeable effect on performance but I wanted to make sure it is on our tracker.

sampottinger commented 11 years ago

Trying to clean up the bug tracker. I don't think this is our priority right now so I am going to kick this down the line a bit.