docker-images icon indicating copy to clipboard operation
docker-images copied to clipboard
verified publisher icon Teradata

Add docker container for MapR

Open AS186170 opened this issue 9 years ago • 9 comments

mapr52-code checkin for docker container

AS186170 avatar Jan 31 '17 19:01 AS186170

Rewrite the commit message. Something like Add docker container for MapR

sanjay990 avatar Jan 31 '17 19:01 sanjay990

Please split the commit in two parts, one for the base image and another one for the -hive one

ArturGajowy avatar Jan 31 '17 21:01 ArturGajowy

Please add a capabilities.txt file so that the tests are run for the -hive image. See any ohter -hive image.

ArturGajowy avatar Jan 31 '17 21:01 ArturGajowy

Guys, any updates on this? Please ping me when the (actionable) comments made so far are applied

cc @petroav @AS186170

ArturGajowy avatar Feb 07 '17 13:02 ArturGajowy

@ArturGajowy i have started working on modification for the review comments today.Will let you know once completed.

AS186170 avatar Feb 13 '17 20:02 AS186170

@sanjay990 @petroav @ArturGajowy Review comments are now complete for the branch

AS186170 avatar Feb 15 '17 16:02 AS186170

Please make the PR only contain two commits, with the respective messages: Add mapr-base image Add mapr-hive image both containing only the changes relevant accordting to their messages

ArturGajowy avatar Feb 22 '17 11:02 ArturGajowy

Please drop '52' from the image & directories names, we're not going to have two versions of the mapr container (say mapr-52 and mapr-53), so there's no point.

ArturGajowy avatar Feb 22 '17 11:02 ArturGajowy

@petroav @ArturGajowy All the review comments are incorporated and all the tests are passing .

AS186170 avatar Apr 25 '17 13:04 AS186170