Re: [rules-dev] DummyKieScanner instantiated in catch{} in Impl

2014-03-10 Thread Mario Fusco
Joe, as said by Mark the DummyKieScanner acts only as a placeholder to avoid tons of null checks inside the code. But you're right when you wrote that it should fail fast when requested to load an artifact. I'll make the DummyKieScanner.loadArtifact methods to throw a more meaningful Exception

Re: [rules-dev] DummyKieScanner instantiated in catch{} in Impl

2014-03-07 Thread Mark Proctor
I suspect the DummyKieScanner was there to avoid people getting null pointers. But I think actually it may be best to throw an exception, to fail fast. The dummy can’t be used for anything, and it will jut confuse users. @mario: thoughts? Mark On 7 Mar 2014, at 23:18, Joe White