barchart / barchart-feed-ddf

barchart data feed libraries
http://www.barchartmarketdata.com/solutions_datafeeds.php
4 stars 2 forks source link

Calling instrument(final String...) with a null entry blocks forever #5

Closed mvberg closed 10 years ago

mvberg commented 10 years ago

Obviously this is user error, but we should be protecting against this.

Looks like the issue could be line 154 in DDF_RxInstrumentProvider

Here is a test case

Iterator<Result<Instrument>> res = marketerDDF.getClient().instrument(new String[] { "AAPL", null }).toBlockingObservable().getIterator();

log.warn("res.hasNext() = " + res.hasNext()); // blocks here
log.warn("res.next() = " + res.next());
GLitchfield commented 10 years ago

Added a defensive check for this