----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62574/#review187091 -----------------------------------------------------------
Thanks for the updated patch however I have noticed that something has changed with the MsSql image thus the start script fails. Please fix it! COMPILING.txt Lines 217 (patched) <https://reviews.apache.org/r/62574/#comment264028> This script causes an ERROR currently: Pulling mssql (microsoft/mssql-server-linux:latest)... ERROR: manifest for microsoft/mssql-server-linux:latest not found - Boglarka Egyed On Oct. 4, 2017, 7:49 a.m., Szabolcs Vasas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62574/ > ----------------------------------------------------------- > > (Updated Oct. 4, 2017, 7:49 a.m.) > > > Review request for Sqoop and Zach Berkowitz. > > > Bugs: SQOOP-3229 > https://issues.apache.org/jira/browse/SQOOP-3229 > > > Repository: sqoop-trunk > > > Description > ------- > > Documentation is added to COMPILING.txt, docker-compose file and helper > scripts are added. > > > Diffs > ----- > > COMPILING.txt eb69b220ecdaedd4d279259aba92df143482145a > scripts/thirdpartytest/docker-compose/db2scripts/db2entrypoint.sh > PRE-CREATION > scripts/thirdpartytest/docker-compose/oraclescripts/healthcheck.sh > PRE-CREATION > > scripts/thirdpartytest/docker-compose/oraclescripts/startup/oracleusersetup.sql > PRE-CREATION > scripts/thirdpartytest/docker-compose/sqoop-thirdpartytest-db-services.yml > PRE-CREATION > scripts/thirdpartytest/start-thirdpartytest-db-containers.sh PRE-CREATION > scripts/thirdpartytest/stop-thirdpartytest-db-containers.sh PRE-CREATION > > > Diff: https://reviews.apache.org/r/62574/diff/3/ > > > Testing > ------- > > > Thanks, > > Szabolcs Vasas > >