Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-29 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/ --- (Updated Jan. 29, 2017, 9:13 a.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/#review163282 --- Ship it! Ship It! - Sebastian Toader On Jan. 27, 2017, 5:12

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/ --- (Updated Jan. 27, 2017, 4:12 p.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/ --- (Updated Jan. 27, 2017, 4:12 p.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/#review163280 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/ --- (Updated Jan. 27, 2017, 4:02 p.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/ --- (Updated Jan. 27, 2017, 2:59 p.m.) Review request for Ambari, Attila Doroszlai,

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Attila Doroszlai
> On Jan. 26, 2017, 9:17 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/resources/scripts/post-user-creation-hook.sh, line 34 > > > > > > I think using `$3` would be better, unless this one will remain the

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-27 Thread Laszlo Puskas
> On Jan. 26, 2017, 8:17 p.m., Attila Doroszlai wrote: > > ambari-server/src/main/resources/scripts/post-user-creation-hook.sh, line 34 > > > > > > I think using `$3` would be better, unless this one will remain the

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-26 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/#review163162 --- Fix it, then Ship it! ambari-server/src/main/java/org/apache/a

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-26 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/#review163142 --- Ship it! Ship It! - Sebastian Toader On Jan. 26, 2017, 6:03

Re: Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-26 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/ --- (Updated Jan. 26, 2017, 5:03 p.m.) Review request for Ambari, Attila Doroszlai,

Review Request 55987: Post user creation hook script creates user home dirs as the configured hdfs_user

2017-01-26 Thread Laszlo Puskas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55987/ --- Review request for Ambari, Attila Doroszlai, Sandor Magyari, and Sebastian Toade