Thanks Taher for your reply. I was just pushing the types to set some type of data as in ignoring. But, now I am completely agree with you on point "The data will automatically get cleaned by committers because no failing data will be committed to the code base".
Again +1 for #3 with option continue-on-failure with default false, with no confusion in thread from my side. :-) -- Rishi Solanki Sr Manager, Enterprise Software Development HotWax Systems Pvt. Ltd. Direct: +91-9893287847 http://www.hotwaxsystems.com On Mon, Jul 10, 2017 at 3:07 PM, Taher Alkhateeb <slidingfilame...@gmail.com > wrote: > Hi Rishi, > > So my suggestion is that if anything does not load, then immediately fail. > > Why am I suggesting this? > - You have to intentionally ignore data failure after being aware of > it (it does not slip between the cracks) > - The data will automatically get cleaned by committers because no > failing data will be committed to the code base. > > I suspect we will actually catch some data loading failures that exist > in the code base and we are maybe unaware of. > > On Mon, Jul 10, 2017 at 12:04 PM, Rishi Solanki <rishisolan...@gmail.com> > wrote: > > I'm good to go with option #3 and continue-on-failure. > > > > Just wanted to mention one thing here; for which type of data we will be > > failing build. That means, we have several options seed, ext, demo. Do we > > need to discuss these points or we are fine for all type of data. Like > demo > > data fails only affect a process for that data set only, and for that > > failing build is okay or not (as on data load we get logs if any file > > didn't load). > > > > > > Btw, I'm good with the proposal, just sharing a thought in case we should > > discuss or may be we can simply ignore if we are good with that. > > > > Thaks! > > > > > > > > Rishi Solanki > > Sr Manager, Enterprise Software Development > > HotWax Systems Pvt. Ltd. > > Direct: +91-9893287847 > > http://www.hotwaxsystems.com > > > > On Mon, Jul 10, 2017 at 2:15 PM, Deepak Dixit < > > deepak.di...@hotwaxsystems.com> wrote: > > > >> > Historically the data loader boolean props are false if ommitted and > the > >> > code expects that, but you have a point about the double negative. We > can > >> > instead call it "continue-on-failure" for example. > >> > > >> > >> +1 continue-on-failure with default value false > >> > >> Thanks & Regards > >> -- > >> Deepak Dixit > >> www.hotwaxsystems.com > >> www.hotwax.co > >> > >> > >> > >> > > >> > On Jul 10, 2017 3:48 AM, "Paul Foxworthy" <p...@cohsoft.com.au> > wrote: > >> > > >> > Hi all, > >> > > >> > I agree with option 3. I recall in my own work I once needed to add a > >> throw > >> > where there was none to track down a problem. > >> > > >> > However ignore-failure leads to a double negative. How about > >> > "stop-on-failure", default value true? > >> > > >> > Cheers > >> > > >> > Paul Foxworthy > >> > > >> > > >> > On 10 July 2017 at 05:27, Taher Alkhateeb <slidingfilame...@gmail.com > > > >> > wrote: > >> > > >> > > Correction: on item (2) in my post: fail immediately, not after > >> > > loading all files, otherwise there's no point. > >> > > > >> > > On Sun, Jul 9, 2017 at 10:18 PM, Taher Alkhateeb > >> > > <slidingfilame...@gmail.com> wrote: > >> > > > Hello Everyone, > >> > > > > >> > > > For a long time I was annoyed by something in OFBiz: the build > system > >> > > > does not fail if data loading fails for some files. I spend hours > >> > > > hunting bugs only to discover that the data simply did not load. > >> > > > > >> > > > Given that I'm working on refactoring the data loading container, > I > >> > > > believe this issue should resolved. However, I'm not sure if the > >> > > > community is interested in making such a change. > >> > > > > >> > > > So I list below 3 options to select from: > >> > > > > >> > > > 1- Leave it as is, do not fail the build if some files do not load > >> > > > 2- Continue loading until all files are done and then fail the > build > >> > > > 3- Provide a flag e.g. ignore-failure that tells the system > whether > >> to > >> > > > fail or not with a default value of "false". > >> > > > > >> > > > My personal preference is for (3) > >> > > > > >> > > > WDYT? > >> > > > >> > > >> > > >> > > >> > -- > >> > Coherent Software Australia Pty Ltd > >> > PO Box 2773 > >> > Cheltenham Vic 3192 > >> > Australia > >> > > >> > Phone: +61 3 9585 6788 > >> > Web: http://www.coherentsoftware.com.au/ > >> > Email: i...@coherentsoftware.com.au > >> > > >> >