suvodeep-pyne commented on pull request #6058:
URL: https://github.com/apache/incubator-pinot/pull/6058#issuecomment-699158595


   @vincentchenjl 
   `persistence-local.yml` is a test resource I believe. I think we need to 
have a way of reading the sql file from a different module. This seems like a 
mvn + macOS issue since the tests run fine on the IDE and on GitHub/Travis.
   
   Moving back the SQL tests is definitely something that we could do but the 
problem is that there are tests which aren't that modular. There may be some 
work here.
   
   Feel free to move the UI app and assets to the `dashboard` module since they 
seem unrelated to the SQL bug. We can tackle this issue separately.


----------------------------------------------------------------
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

Reply via email to