Github user DaveBirdsall commented on a diff in the pull request:
https://github.com/apache/incubator-trafodion/pull/566#discussion_r69046017
--- Diff: install/installer/traf_cloudera_mods ---
@@ -367,27 +370,37 @@ ssh -q -n $HDFS_NODE 'sudo su hdfs --command "'
"$HADOOP_BIN_PATH"'/hdfs dfsadmi
# enabled and HDFS has been restarted
echo "***INFO: Setting HDFS ACLs for snapshot scan support"
ssh -q -n $HDFS_NODE 'sudo su hdfs --command "' "$HADOOP_BIN_PATH"'/hdfs
dfs -mkdir -p /hbase/archive"'
-if [ $? != 0 ]; then
+errorFound=$?
+echo "ERROR Code: $errorFound"
--- End diff --
Do you want this echo here? Users will see a series of "ERROR Code: 0"
messages in normal circumstances and wonder what all the fuss is about. Perhaps
the value of $errorFound should be displayed inside the 'if' following?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
