mcvsubbu commented on issue #5559: URL: https://github.com/apache/incubator-pinot/issues/5559#issuecomment-649672637
> whats the fix? Good question. I don't have an answer at the moment. As a reviewer, the best I can do (and sometimes it is not possible either) to look at problems it may bring about. Keeping the truncate facility as a command line argument will help, because we will NOT be running validation manager, etc. We can check things all we want, report error to the user, user can run it again, etc. Why can't we do that ? Providing primitive APIs, and command lines on top of that is a good practice for most things, especially this one. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
