liyafan82 commented on pull request #10789: URL: https://github.com/apache/arrow/pull/10789#issuecomment-922324693
> > > Can you confirm if the new app can be integrates with the upstream fuzz project? > > > > > > After some attempts, I do not feel it a good idea to run the test from the command line, as it requires too many dependencies. > > So I have transformed the test into a UT (the running time is not long). > > Sorry I missed this response. I think integrating it with the upstream fuzzing library is important because it means it will get tested against new random inputs that might expose issues in the future. What do you mean by too many dependencies? @emkornfield Sorry for my late reply. I am tied up to work and have very limited bandwidth these days. > I think integrating it with the upstream fuzzing library is important because it means it will get tested against new random inputs that might expose issues in the future. I agree with you, and I do not think the current implementation (do it through UT) contradicts with this objective, as new inputs will be tested automatically after being added to the repository. Maybe I misunderstand your point? > What do you mean by too many dependencies? If we run the test from command line, we will find that it requires too many jar libraries on the class path (some are directly referenced, while others are indirectly referenced). This makes the command extremely long (maybe it spans a few pages) and ugly. This can be easily verified, as some IDE (e.g. Intellij) prints the command line in the console for each UT/Application being run. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
