search

pilwon / node-yahoo-finance

Yahoo Finance historical quotes and snapshot data downloader written in Node.js
491 stars 123 forks source link

Promise based changes in historical and snapshot #19

Closed yagop closed 8 years ago

yagop commented 8 years ago

Removed unnecessary Promise.resolve()

Added options.maxConcurrentSymbols to specify the number of requests as the same time. If not set, use os.cpus().length but it would be better to increase that number. The CPU can handle much more Promises than its cores. The bottleneck is on the HTTP requests time where could be very lazy dealing with hundred of symbols.

yagop commented 8 years ago

ping

mazing commented 8 years ago

+1

pilwon commented 8 years ago

@yagop Thanks-