Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11731 )

Change subject: IMPALA-7698: Add centos support to bootstrap_system.
......................................................................


Patch Set 1:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/11731/1/bin/bootstrap_system.sh
File bin/bootstrap_system.sh:

http://gerrit.cloudera.org:8080/#/c/11731/1/bin/bootstrap_system.sh@131
PS1, Line 131:  psmisc lsof sudo openssh-server redhat-lsb 
java-1.8.0-openjdk-devel \
tab used for whitespace


http://gerrit.cloudera.org:8080/#/c/11731/1/bin/bootstrap_system.sh@143
PS1, Line 143: redhat sudo wget -nv 
http://www-us.apache.org/dist/maven/maven-3/3.5.4/binaries/apache-maven-3.5.4-bin.tar.gz
 http://www-us.apache.org/dist/ant/binaries/apache-ant-1.9.13-bin.tar.gz
line too long (181 > 90)


http://gerrit.cloudera.org:8080/#/c/11731/1/bin/bootstrap_system.sh@191
PS1, Line 191: ubuntu sudo sed -ri 's/local +all +all +peer/local all all 
trust/g' /etc/postgresql/*/main/pg_hba.conf
line too long (102 > 90)


http://gerrit.cloudera.org:8080/#/c/11731/1/bin/bootstrap_system.sh@192
PS1, Line 192: redhat sudo sed -ri 's/local +all +all +ident/local all all 
trust/g' /var/lib/pgsql/data/pg_hba.conf
line too long (100 > 90)


http://gerrit.cloudera.org:8080/#/c/11731/1/bin/bootstrap_system.sh@194
PS1, Line 194: # redhat sudo bash -c "echo host all all 127.0.0.1/32 md5 >> 
/var/lib/pgsql/data/pg_hba.conf"
line too long (93 > 90)


http://gerrit.cloudera.org:8080/#/c/11731/1/bin/bootstrap_system.sh@257
PS1, Line 257: redhat sudo sed -i 's,\*\s*soft\s*nproc\s*1024,* soft nproc 
unlimited,' /etc/security/limits.d/90-nproc.conf
line too long (108 > 90)



--
To view, visit http://gerrit.cloudera.org:8080/11731
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id54294d7607f51de87a9de373dcfc4a33f4bedf5
Gerrit-Change-Number: 11731
Gerrit-PatchSet: 1
Gerrit-Owner: Philip Zeyliger <phi...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Laszlo Gaal <laszlo.g...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>
Gerrit-Comment-Date: Fri, 19 Oct 2018 03:19:45 +0000
Gerrit-HasComments: Yes

Reply via email to