maytasm commented on pull request #10667: URL: https://github.com/apache/druid/pull/10667#issuecomment-742839022
> > @capistrant Have you run these tests? (Or do they run in Travis CI automatically, which would be even better?) > > If these changes work properly, then they look good to me. The additional docs are helpful, too. > > None of the hadoop related integration test groups are a part of CI. Why that is, I am not totally sure. > > I have flexed the new argument for mvn locally though and run the ITHadoopIndex test when using it. Seems to work. But thats really the only material change and since no hadoop stuff is involved with TravisCI at this time, CI shouldn't be impacted at all. Hadoop IT requires a lot of memory. Locally, I give Docker around 11GB of memory. I believe anything less than 8-9GB will cause the cluster to fail when you also run hadoop container. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
