[GitHub] incubator-trafodion pull request #1315: additional fixes to [TRAFODION-2806]

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1315

additional fixes to [TRAFODION-2806]

sometimes the hbase classpath may get updated,  the cache file will be 
obsolete. Now we clean the cache file during 'sqstart/sqgen'

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2806_cont

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1315.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1315


commit 7920ef32744e7c5ac65dd605065b505788668a77
Author: Eason <hfutm...@gmail.com>
Date:   2017-11-29T10:10:34Z

additional fixes to [TRAFODION-2806]




---

[GitHub] incubator-trafodion pull request #1299: [TRAFODION-2806] optimize 'hbase cla...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1299

[TRAFODION-2806] optimize 'hbase classpath' cmd in sqenvcom.sh

cache the output of 'hbase classpath' to a file under 
$TRAF_VAR/hbase_classpath, then sqenvcom.sh will get the hbase lib from this 
file if exists.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2806

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1299.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1299


commit 2d9aaa2302f4f747d6095a3c33fb452920500ea8
Author: Eason <hfutm...@gmail.com>
Date:   2017-11-15T02:29:58Z

[TRAFODION-2806] optimize 'hbase classpath' cmd in sqenvcom.sh




---

[GitHub] incubator-trafodion pull request #1152: [TRAFODION-2658] pyinstaller - suppo...

[mkby]

Github user mkby commented on a diff in the pull request:


https://github.com/apache/incubator-trafodion/pull/1152#discussion_r125371611
  
--- Diff: core/sqf/sql/scripts/sqregen ---
@@ -41,8 +41,15 @@ function Usage {
 
 function RegenConfig {
 echo "Regenerating Trafodion configuration: $SQCONFIG_FILE"
+cd $TRAF_HOME/sql/scripts
 ./regensq.pl $SQCONFIG_FILE
-exit $?;
+if [ $? != 0 ]; then
+echo "Failed to run regensq.pl."
+exit $?
+fi
+echo "Copying Trafodion configuration file $SQCONFIG_FILE to all nodes"
+pdcp `trafconf -wname` $SQCONFIG_FILE $TRAF_HOME/sql/scripts/
--- End diff --

I think pdcp itself will show the error message if fails


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1152: [TRAFODION-2658] pyinstaller - suppo...

[mkby]

Github user mkby commented on a diff in the pull request:


https://github.com/apache/incubator-trafodion/pull/1152#discussion_r125371556
  
--- Diff: core/sqf/sql/scripts/sqregen ---
@@ -41,8 +41,15 @@ function Usage {
 
 function RegenConfig {
 echo "Regenerating Trafodion configuration: $SQCONFIG_FILE"
+cd $TRAF_HOME/sql/scripts
 ./regensq.pl $SQCONFIG_FILE
-exit $?;
+if [ $? != 0 ]; then
+echo "Failed to run regensq.pl."
--- End diff --

sounds good, I will change the message, thanks.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1155: [TRAFODION-2664] Instance will be do...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1155

[TRAFODION-2664] Instance will be down when the zookeeper on name node has 
been down

do retry on every 'zoo_exists' call when we get a ZCONNECTIONLOSS failure.
@zcorrea , could you have to review the changes?

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2664

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1155.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1155


commit 7e1408c5584ae587e4d1198fa0bcf95a4c43c2cc
Author: Eason <hfutm...@gmail.com>
Date:   2017-06-29T02:25:07Z

[TRAFODION-2664] Instance will be down when the zookeeper on name node has 
been down




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1152: [TRAFODION-2658] pyinstaller - suppo...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1152

[TRAFODION-2658] pyinstaller - support elastic adding nodes

init version of pyinstaller elastic feature.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion pyinstall_elastic

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1152.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1152


commit d07a3042595b9af4b72b7caea9782f969fa54b25
Author: Eason <hfutm...@gmail.com>
Date:   2017-06-21T13:10:04Z

[TRAFODION-2658] pyinstaller - support elastic adding nodes




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1106: rework of TRAFODION-2001

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1106

rework of TRAFODION-2001

**rework jira 1860:**

missing ctime.pl file on CentOS7.

[trafodion@vmcentos7 ~]$ sqgen
Workstation environment - Not a clustered environment

Can't locate ctime.pl in @INC (@INC contains: 
/home/trafodion/trafodion-2.0.1/export/lib /usr/local/lib64/perl5 
/usr/local/share/perl5 /usr/lib64/perl5/vendor_perl 
/usr/share/perl5/vendor_perl /usr/lib64/perl5 /usr/share/perl5 .) at ./gensq.pl 
line 24.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion rework2001

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1106.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1106


commit 0530f22bc3aca8c011d96ce2c9c5eb61e0688082
Author: Eason <hfutm...@gmail.com>
Date:   2017-06-01T03:00:56Z

rework of TRAFODION-2001




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1105: [TRAFODION-2629] pyinstaller detects...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1105

[TRAFODION-2629] pyinstaller detects CDH parcel incorrectly.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2629

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1105.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1105


commit 3ed2d88bed63226b0caffc0b8abd90dc4a5fb4ff
Author: Eason <hfutm...@gmail.com>
Date:   2017-05-31T05:44:14Z

[TRAFODION-2629] pyinstaller detects CDH parcel incorrectly.




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1084: [TRAFODION-2607] pyinstaller - donno...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1084

[TRAFODION-2607] pyinstaller - donnot convert hostname to short name



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2607

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1084.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1084


commit b49f15919735a63b2df138ed545b8daa534ab00d
Author: Eason <hfutm...@gmail.com>
Date:   2017-05-08T14:36:21Z

[TRAFODION-2607] pyinstaller - donnot convert hostname to short name




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1077: Trafodion 2001

[mkby]

Github user mkby commented on a diff in the pull request:


https://github.com/apache/incubator-trafodion/pull/1077#discussion_r115188436
  
--- Diff: core/sqf/sql/scripts/sqnodestatus ---
@@ -26,12 +26,11 @@
 # Implements:
 #   ssh -M "ConnectTimeout 1" -o "ConnectionAttempts 3" n028 hostinfo 2>&1
 #
-# Uses:
-#   export MY_NODES=' -w n028 -w n044 -w n029 -w n045 -w n030 -w n031 -w 
n032 '
+# Uses the following enviroment variables:
 #   export SQ_MON_SSH_OPTIONS=' -o "ConnectTimeout 1" -o 
"ConnectionAttempts 3" '
 #
 
-my $node_context=lc(readpipe("echo \$NODE_LIST"));
+my $node_context=readpipe("trafconf -name");
--- End diff --

looks good!


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1077: Trafodion 2001

[mkby]

Github user mkby commented on a diff in the pull request:


https://github.com/apache/incubator-trafodion/pull/1077#discussion_r115133212
  
--- Diff: core/sqf/sql/scripts/sqnodestatus ---
@@ -26,12 +26,11 @@
 # Implements:
 #   ssh -M "ConnectTimeout 1" -o "ConnectionAttempts 3" n028 hostinfo 2>&1
 #
-# Uses:
-#   export MY_NODES=' -w n028 -w n044 -w n029 -w n045 -w n030 -w n031 -w 
n032 '
+# Uses the following enviroment variables:
 #   export SQ_MON_SSH_OPTIONS=' -o "ConnectTimeout 1" -o 
"ConnectionAttempts 3" '
 #
 
-my $node_context=lc(readpipe("echo \$NODE_LIST"));
+my $node_context=readpipe("trafconf -name");
--- End diff --

will `trafconf -name` return the lowercase hostname or the upper/lower case 
of hostname is addressed in other codes?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1061: [TRAFODION-2588] monitor failed to s...

[mkby]

Github user mkby commented on a diff in the pull request:


https://github.com/apache/incubator-trafodion/pull/1061#discussion_r111298734
  
--- Diff: core/sqf/monitor/linux/zclient.cxx ---
@@ -53,6 +53,9 @@
 // The monitors register their znodes under the cluster znode
 #define ZCLIENT_CLUSTER_ZNODE   "/cluster"
 
+// zookeeper connection retry count
+#define ZOOKEEPER_RETRY_COUNT3
--- End diff --

Based on the test result, actually retry for once is enough for this case. 
So it's not quite necessary to modify this value. 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1062: [TRAFODION-2589] pyinstaller - do re...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1062

[TRAFODION-2589] pyinstaller - do remote node initialize in parallel



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2589

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1062.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1062


commit f45c3aa6a541445142bddca9d18bf93d78458d40
Author: Eason <hfutm...@gmail.com>
Date:   2017-04-11T10:11:48Z

[TRAFODION-2589] pyinstaller - do remote node initialize in parallel




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1061: [TRAFODION-2588] monitor failed to s...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1061

[TRAFODION-2588] monitor failed to start if part of zookeeper server is down

retry 3 times when connecting to zookeeper.

@narendragoyal @zcorrea please review.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2588

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1061.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1061


commit 18ad65a99275b44a3f68db01ca5385665a2e5195
Author: Eason <hfutm...@gmail.com>
Date:   2017-04-11T09:58:32Z

[TRAFODION-2588] monitor failed to start if part of zookeeper server is down




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1049: [TRAFODION-2579] installers should a...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1049

[TRAFODION-2579] installers should allow multiple ldap hosts and ldap UID 
lines

changes for pyinstaller part.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2579

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1049.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1049


commit 4de192332ef35768f8a86890c6a765b3df692999
Author: Eason <hfutm...@gmail.com>
Date:   2017-04-06T08:24:53Z

[TRAFODION-2579] installers should allow multiple ldap hosts and ldap UID 
lines




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1048: [TRAFODION-2526] pyinstaller - don't...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1048

[TRAFODION-2526] pyinstaller - don't use sudo command in root user during 
installation

along with some other stability fixes.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2526

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1048.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1048


commit 34c70d0d3bc7c8d11310719030ddc48768df5c07
Author: Eason <hfutm...@gmail.com>
Date:   2017-04-06T07:29:51Z

[TRAFODION-2526] pyinstaller - don't use sudo command in root user during 
installation




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1042: Rework PR-1009 for pyinstaller confi...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1042

Rework PR-1009 for pyinstaller config change



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion patch-1

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1042.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1042


commit cfda8c1d00fb7191fbcd9326a03f6ae0769f8974
Author: Eason <hfutm...@gmail.com>
Date:   2017-04-02T12:07:44Z

Rework PR-1009 for pyinstaller config change




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1015: [TRAFODION-2526] pyinstaller - don't...

[mkby]

Github user mkby closed the pull request at:

https://github.com/apache/incubator-trafodion/pull/1015


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1038: cherry-pick pyinstaller changes from...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1038

cherry-pick pyinstaller changes from release2.1 to master



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion mrg_installer

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1038.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1038


commit eb4b0a3149aec9190a8cf53de91709cea6368ce8
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-23T08:04:48Z

[TRAFODION-2496] pyinstaller - better handle constants

commit dac0a44ef12e5c3aa485f110fcbdfb786876f1cc
Author: Eason <hfutm...@gmail.com>
Date:   2017-03-16T09:22:39Z

[TRAFODION-2539] pyinstaller - set actual trafodion group if trafodion user 
exists

commit 9542a8fe4a63f3f56323575b01c925c7b823bdac
Author: Eason <hfutm...@gmail.com>
Date:   2017-03-13T05:31:18Z

[TRAFODION-2533] pyinstaller - check ssh pam settings during installation

commit 71f0a62a05f4de8ec48fb3f7ad8a9bb341804182
Author: Eason <hfutm...@gmail.com>
Date:   2017-03-08T02:28:34Z

[TRAFODION-2525] pyinstaller - support HBase1.2 version in config file

commit 6a65dcab0a8b045385c3147a89e873e7be6a465f
Author: Eason <hfutm...@gmail.com>
Date:   2017-03-07T03:21:23Z

optimize ps command in uninstall script

commit eaa45f6f52e93c4ef5030baa447bde715cce8f3e
Author: Eason <hfutm...@gmail.com>
Date:   2017-03-06T16:28:09Z

[TRAFODION-2518] pyinstaller - remove unused db_admin settings.

commit 40d3352d6e439aee0b5e319b5b6fac39bb6ca9be
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-27T02:14:59Z

[TRAFODION-2510] pyinstaller - remove DB__ADMIN from installer

commit ce61a280ec8fc5467b0b90f98794f82be3c61381
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-24T04:52:46Z

[TRAFODION-2482] documentation for python installer




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1025: [TRAFODION-2532] clean up the error ...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1025

[TRAFODION-2532] clean up the error message 'monitor.ini not found'

roughly clean up codes releated to TmSeq and monitor.ini, not quite sure 
everything is good, @narendragoyal please take a look, thanks.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2532

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1025.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1025


commit c2704e71d02e459236ae26c81a001b0231598067
Author: Eason <hfutm...@gmail.com>
Date:   2017-03-27T14:14:57Z

[TRAFODION-2532] clean up the error message 'monitor.ini not found'




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1015: [TRAFODION-2526] pyinstaller - don't...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1015

[TRAFODION-2526] pyinstaller - don't use sudo command in root user during 
installation



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion r21-2526

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1015.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1015


commit 8046e4505e509ae22ecdbefda5d717ba293b7d2d
Author: Eason <hfutm...@gmail.com>
Date:   2017-03-16T10:19:32Z

[TRAFODION-2526] pyinstaller - don't use sudo command in root user during 
installation




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1014: [TRAFODION-2539] pyinstaller - set a...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1014

[TRAFODION-2539] pyinstaller - set actual trafodion group if trafodion user 
exists



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion r21-2539

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1014.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1014


commit 96bf15cfd1538f7e1e324d97e1aa6f43f962a0dc
Author: Eason <hfutm...@gmail.com>
Date:   2017-03-16T09:22:39Z

[TRAFODION-2539] pyinstaller - set actual trafodion group if trafodion user 
exists




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #1005: [TRAFODION-2533] pyinstaller - check...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/1005

[TRAFODION-2533] pyinstaller - check ssh pam settings during installation



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion r21-2533

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/1005.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1005


commit 5d35795061af9c06bc1d1715d28e951ba3977546
Author: Eason <hfutm...@gmail.com>
Date:   2017-03-13T05:31:18Z

[TRAFODION-2533] pyinstaller - check ssh pam settings during installation




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #999: [TRAFODION-2525] pyinstaller - suppor...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/999

[TRAFODION-2525] pyinstaller - support HBase1.2 version in config file



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion r21-2525

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/999.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #999


commit 8b9b21425f55fa2929d5d2f79b67a9fe04865f38
Author: Eason <hfutm...@gmail.com>
Date:   2017-03-08T02:28:34Z

[TRAFODION-2525] pyinstaller - support HBase1.2 version in config file




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #991: [TRAFODION-2518] pyinstaller - remove...

[mkby]

Github user mkby commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/991#discussion_r104580774
  
--- Diff: install/python-installer/db_uninstall.py ---
@@ -96,13 +96,13 @@ def main():
 
 remotes = [Remote(node, pwd=pwd) for node in nodes]
 # stop trafodion on the first node
-remotes[0].execute('sudo su %s -l -c ckillall' % TRAF_USER, 
chkerr=False)
+#remotes[0].execute('sudo su %s -l -c ckillall' % TRAF_USER, 
chkerr=False)
 
 # remove trafodion userid and group on all trafodion nodes, together 
with folders
 for remote in remotes:
 info('Remove Trafodion on node [%s] ...' % remote.host)
 remote.execute('ps -ef|grep ^%s|awk \'{print $2}\'|xargs sudo kill 
-9' % TRAF_USER, chkerr=False)
-remote.execute('ps -ef|grep krb5check|awk \'{print $2}\'|xargs 
sudo kill -9', chkerr=False) # traf kerberos service
+remote.execute('trafid=`getent passwd %s|awk -F: \'{print $3}\'`; 
if [[ -n $trafid ]]; then ps -ef|grep ^$trafid|awk \'{print $2}\'|xargs sudo 
kill -9; fi' % TRAF_USER, chkerr=False)
 remote.execute('sudo -n /usr/sbin/userdel -rf %s' % TRAF_USER, 
chkerr=False)
--- End diff --

good suggestion, thanks.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #991: [TRAFODION-2518] pyinstaller - remove...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/991

[TRAFODION-2518] pyinstaller - remove unused db_admin settings.

[TRAFODION-2519] pyinstaller - uninstaller should kill all trafodion 
processes.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion r21-2518

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/991.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #991


commit dc6e7bd4081cdf885cba3708c9d7cf727515a6c4
Author: Eason <hfutm...@gmail.com>
Date:   2017-03-06T16:28:09Z

[TRAFODION-2518] pyinstaller - remove unused db_admin settings.




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #980: [TRAFODION-2510] pyinstaller - remove...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/980

[TRAFODION-2510] pyinstaller - remove DB__ADMIN from installer



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion r21-2510

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/980.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #980


commit 476519d2b983c22a8d0d56998e5fd14ddf83a8c8
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-27T02:14:59Z

[TRAFODION-2510] pyinstaller - remove DB__ADMIN from installer




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #973: [TRAFODION-2482] documentation for py...

[mkby]

Github user mkby commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/973#discussion_r102673703
  
--- Diff: docs/provisioning_guide/src/asciidoc/_chapters/quickstart.adoc ---
@@ -48,60 +48,55 @@ NOTE: You can download and install the {project-name} 
Clients once you've instal
 
 *Example*
 
+Download the Trafodion Installer and Server binaries:
 ```
 $ mkdir $HOME/trafodion-download
 $ cd $HOME/trafodion-download
 $ # Download the Trafodion Installer binaries
-$ wget 
http://apache.cs.utah.edu/incubator/trafodion/trafodion-1.3.0.incubating/apache-trafodion-installer-1.3.0-incubating-bin.tar.gz
+$ wget 
http://apache.cs.utah.edu/incubator/trafodion/trafodion-2.1.0.incubating/apache-trafodion-pyinstaller-2.1.0-incubating.tar.gz
 Resolving http://apache.cs.utah.edu... 192.168.1.56
 Connecting to http://apache.cs.utah.edu|192.168.1.56|:80... connected.
 HTTP request sent, awaiting response... 200 OK
 Length: 68813 (67K) [application/x-gzip]
-Saving to: "apache-trafodion-installer-1.3.0-incubating-bin.tar.gz"
+Saving to: "apache-trafodion-pyinstaller-2.1.0-incubating.tar.gz"
 
 
100%[=>]
 68,813   124K/s   in 0.5s
 
-2016-02-14 04:19:42 (124 KB/s) - 
"apache-trafodion-installer-1.3.0-incubating-bin.tar.gz" saved [68813/68813]
-```
+2016-02-14 04:19:42 (124 KB/s) - 
"apache-trafodion-pyinstaller-2.1.0-incubating.tar.gz" saved [68813/68813]
 
-<<<
-```
-$ # Download the Trafodion Server binaries
-$ wget 
http://apache.cs.utah.edu/incubator/trafodion/trafodion-1.3.0.incubating/apache-trafodion-1.3.0-incubating-bin.tar.gz
+$ wget 
http://apache.cs.utah.edu/incubator/trafodion/trafodion-2.1.0.incubating/apache-trafodion_server-2.1.0-RH-x86_64-incubating.tar.gz
 Resolving http://apache.cs.utah.edu... 192.168.1.56
 Connecting to http://apache.cs.utah.edu|192.168.1.56|:80... connected.
 HTTP request sent, awaiting response... 200 OK
 Length: 214508243 (205M) [application/x-gzip]
-Saving to: "apache-trafodion-1.3.0-incubating-bin.tar.gz"
+Saving to: "apache-trafodion_server-2.1.0-RH-x86_64-incubating.tar.gz"
 
--- End diff --

This is the package name after `make package` on my system, is there 
anything wrong with my environment?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #978: [TRAFODION-2496] pyinstaller - better...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/978

[TRAFODION-2496] pyinstaller - better handle constants

Put constants together into a new constants.py file. It will be easy to 
modify them to fit new Hadoop environment.

along with some minor fixes:
1. kill krb5service and delete status file in uninstall script
2. fix hbase_lib_path issue while install trafodion on apache hadoop
3. typo in db_config_default

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion r21-2496

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/978.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #978


commit dc58c85d3b0c7364461129ac7973c5f97b6576a8
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-23T08:04:48Z

[TRAFODION-2496] pyinstaller - better handle constants




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #973: [TRAFODION-2482] documentation for py...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/973

[TRAFODION-2482] documentation for python installer



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion r21-2482

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/973.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #973


commit c155bfaecfcb93755e401ee0551e32b50ec7b370
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-21T09:29:42Z

[TRAFODION-2482] documentation for python installer




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #969: [TRAFODION-2489] [TRAFODION-2490] fix...

[mkby]

Github user mkby closed the pull request at:

https://github.com/apache/incubator-trafodion/pull/969


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #970: [TRAFODION-2489] [TRAFODION-2490] fix...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/970

[TRAFODION-2489] [TRAFODION-2490] fix for pyinstaller

[TRAFODION-2489] pyinstaller - sqconfig on single node
[TRAFODION-2490] pyinstaller - admin user setup gives syntax error

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion r21-2490

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/970.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #970


commit 63f947c77c3a250a760be44b13a964319074e9e7
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-20T06:02:04Z

[TRAFODION-2489] pyinstaller - sqconfig on single node
[TRAFODION-2490] pyinstaller - admin user setup gives syntax error




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #956: [TRAFODION-2480] monitor should not c...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/956

[TRAFODION-2480] monitor should not check hostname strictly

cherry pick from esgyndb.

@narendragoyal , please review.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2480

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/956.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #956


commit 001cc3a01d79a0ba91dfc5ce51dfea243fc33c52
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-13T07:22:42Z

[TRAFODION-2480] monitor should not check hostname strictly




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #955: [TRAFODION-2479] pyinstaller - firewa...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/955

[TRAFODION-2479] pyinstaller - firewall status should not block the 
installation process

@svarnau , please take a look

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2479

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/955.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #955


commit 5b86a0625913f1b811915e370053a08673e75550
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-13T03:18:26Z

[TRAFODION-2479] pyinstaller - firewall status should not block the 
installation process




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #954: [TRAFODION-2476] pyinstaller - error ...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/954

[TRAFODION-2476] pyinstaller - error in DCS HA configuration



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2476

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/954.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #954


commit bd21beb5fe77ba394cd5029062d6378505906fc7
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-10T15:43:47Z

[TRAFODION-2476] pyinstaller - error in DCS HA configuration




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #952: [TRAFODION-2473] sqnodestatus report ...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/952

[TRAFODION-2473] sqnodestatus report wrong status if ssh banner is 
configured.

specify the '-q' command to ignore ssh banner.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2473

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/952.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #952


commit a265a2176c4189d5ba9fb49080f2cb2f8f06c310
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-10T13:35:12Z

[TRAFODION-2473] sqnodestatus report wrong status if ssh banner is 
configured.




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #950: [TRAFODION-2471] pyinstaller: error c...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/950

[TRAFODION-2471] pyinstaller: error command when grant privilege in HBase.

The old command doesn't work.
also change the owner of HDFS directory /user/trafodion

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2471

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/950.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #950


commit 0c3cdbdf0f487c79728fdf1e17a63b9e3c07b571
Author: Eason <hfutm...@gmail.com>
Date:   2017-02-09T06:48:10Z

[TRAFODION-2471] pyinstaller: error command when grant privilege in HBase.




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #927: [TRAFODION-2460] pyinstaller-should n...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/927

[TRAFODION-2460] pyinstaller-should not overwrite previous 
hbase.coprocessor.region.class settings when modify HDP settings.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2460

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/927.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #927


commit 7c3ae9655c213c641aad2ba4ea214c7b16821bd4
Author: Eason <hfutm...@gmail.com>
Date:   2017-01-25T13:32:23Z

[TRAFODION-2460] pyinstaller-should not overwrite previous 
hbase.coprocessor.region.class settings when modify HDP settings.




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #920: [TRAFODION-2454] pyinstaller: don't r...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/920

[TRAFODION-2454] pyinstaller: don't restart all hdp services



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2454

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/920.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #920


commit 6ac3516adc0723b074b826c3e8e4728d4b599fd7
Author: Eason <hfutm...@gmail.com>
Date:   2017-01-19T16:32:32Z

[TRAFODION-2454] pyinstaller: don't restart all hdp services




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #919: [TRAFODION-2453] pyinstaller error on...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/919

[TRAFODION-2453] pyinstaller error on hdp with kerberos enabled.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2453

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/919.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #919






---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #912: [TRAFODION-2450] copy file error for ...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/912

[TRAFODION-2450] copy file error for python-installer/templates

remove unused templates folder.
copy bashrc file in 'traf_setup' step, because trafodion package is not 
extracted in 'traf_user' step.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2450

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/912.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #912


commit 51d3c625d031e455ebd3ca1b6eaad47b9151083d
Author: Eason <hfutm...@gmail.com>
Date:   2017-01-16T15:58:43Z

[TRAFODION-2450] copy file error for python-installer/templates




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #901: [TRAFODION-2291] Ambari integration

[mkby]

Github user mkby commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/901#discussion_r95057223
  
--- Diff: install/README.md ---
@@ -0,0 +1,75 @@
+
+
+## Trafodion Installers
+
+* **install** - This is the current command-line installer. It installs a 
server tarball
+  on an existing CDH or HDP Hadoop cluster.
+* **python-installer** - This is the new command-line installer, meant to 
replace current
+  command-line installer. Likewise, installs server tarball on existing 
CDH, HDP,
+  or APACHE cluster.
+* **ambari-installer** - This integrates with Ambari cluster manager, so 
only applies to HDP.
+  In this case, trafodion server is installed via RPM. This is installed 
on Ambari server as
+  a management pack. Trafodion can be included in the initial cluster 
creation or added later.
+
+## Ambari Integration
+
+The Ambari MPack (management pack) is also packaged as an RPM, having a 
dependency on ambari-server.
+Given a proper yum repo file, `traf_ambari` rpm
+can be installed directly and it pulls in ambari-server.
+If ambari-server is previously installed and running, it must be restarted 
to pick up the Trafodion
+management pack.
+
+ Packaging
+
+Part of Ambari's job is to set up yum repo files on each node in order to 
install packages.
+The default URLs are for Hortonworks' public repos. But since your 
custom-built Trafodion is
+not hosted there, you need to specify a URL for your local yum repo 
server. To build that into
+the `traf_ambari` package, use make to specify value of `REPO_URL`.
+
+   `make package REPO_URL=http://my.repo.server/repo/...`
--- End diff --

Great! So I think `REPO_URL` should only be the default value in the GUI, 
and user can modify it. It's acceptable to me.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #901: [TRAFODION-2291] Ambari integration

[mkby]

Github user mkby commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/901#discussion_r94922018
  
--- Diff: install/README.md ---
@@ -0,0 +1,75 @@
+
+
+## Trafodion Installers
+
+* **install** - This is the current command-line installer. It installs a 
server tarball
+  on an existing CDH or HDP Hadoop cluster.
+* **python-installer** - This is the new command-line installer, meant to 
replace current
+  command-line installer. Likewise, installs server tarball on existing 
CDH, HDP,
+  or APACHE cluster.
+* **ambari-installer** - This integrates with Ambari cluster manager, so 
only applies to HDP.
+  In this case, trafodion server is installed via RPM. This is installed 
on Ambari server as
+  a management pack. Trafodion can be included in the initial cluster 
creation or added later.
+
+## Ambari Integration
+
+The Ambari MPack (management pack) is also packaged as an RPM, having a 
dependency on ambari-server.
+Given a proper yum repo file, `traf_ambari` rpm
+can be installed directly and it pulls in ambari-server.
+If ambari-server is previously installed and running, it must be restarted 
to pick up the Trafodion
+management pack.
+
+ Packaging
+
+Part of Ambari's job is to set up yum repo files on each node in order to 
install packages.
+The default URLs are for Hortonworks' public repos. But since your 
custom-built Trafodion is
+not hosted there, you need to specify a URL for your local yum repo 
server. To build that into
+the `traf_ambari` package, use make to specify value of `REPO_URL`.
+
+   `make package REPO_URL=http://my.repo.server/repo/...`
--- End diff --

thanks, sounds good to me


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #906: [TRAFODION-2430] python installer - n...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/906

[TRAFODION-2430] python installer - need to modify Makefile for building a 
tar package



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2430

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/906.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #906


commit a5dd41d80c7f05b1d4c0148a4f86a77ef7758798
Author: Eason <hfutm...@gmail.com>
Date:   2017-01-06T06:35:32Z

[TRAFODION-2430] python installer - need to modify Makefile for building a 
tar package




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #905: [TRAFODION-2429] python installer - r...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/905

[TRAFODION-2429] python installer - remove all templates file

along with some other tiny changes.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2429

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/905.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #905


commit ba6136d13a172f5b2ac02345189a29964b2a74d7
Author: Eason <hfutm...@gmail.com>
Date:   2017-01-06T06:09:12Z

[TRAFODION-2429] python installer - remove all templates file




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #901: [TRAFODION-2291] Ambari integration

[mkby]

Github user mkby commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/901#discussion_r94902248
  
--- Diff: install/README.md ---
@@ -0,0 +1,75 @@
+
+
+## Trafodion Installers
+
+* **install** - This is the current command-line installer. It installs a 
server tarball
+  on an existing CDH or HDP Hadoop cluster.
+* **python-installer** - This is the new command-line installer, meant to 
replace current
+  command-line installer. Likewise, installs server tarball on existing 
CDH, HDP,
+  or APACHE cluster.
+* **ambari-installer** - This integrates with Ambari cluster manager, so 
only applies to HDP.
+  In this case, trafodion server is installed via RPM. This is installed 
on Ambari server as
+  a management pack. Trafodion can be included in the initial cluster 
creation or added later.
+
+## Ambari Integration
+
+The Ambari MPack (management pack) is also packaged as an RPM, having a 
dependency on ambari-server.
+Given a proper yum repo file, `traf_ambari` rpm
+can be installed directly and it pulls in ambari-server.
+If ambari-server is previously installed and running, it must be restarted 
to pick up the Trafodion
+management pack.
+
+ Packaging
+
+Part of Ambari's job is to set up yum repo files on each node in order to 
install packages.
+The default URLs are for Hortonworks' public repos. But since your 
custom-built Trafodion is
+not hosted there, you need to specify a URL for your local yum repo 
server. To build that into
+the `traf_ambari` package, use make to specify value of `REPO_URL`.
+
+   `make package REPO_URL=http://my.repo.server/repo/...`
--- End diff --

@svarnau, thanks for the explanation. It seems that traf-ambari rpm package 
is customized with different REPO_URL, so everytime we install on a different 
envionment, we have to build a new traf-ambari rpm to apply the new URL, is 
that true? 
Can we put this URL into ambari webpage so we can configure the URL  from 
there? 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #902: [TRAFODION-2428] bash installer: alwa...

[mkby]

Github user mkby closed the pull request at:

https://github.com/apache/incubator-trafodion/pull/902


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #902: [TRAFODION-2428] bash installer: alwa...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/902

[TRAFODION-2428] bash installer: always copy bashrc template file to …

…trafodion's $HOME

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2428

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/902.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #902


commit dd4b16808d1fabe1ea84eadffc8807b9ac937968
Author: Eason <hfutm...@gmail.com>
Date:   2017-01-05T14:53:50Z

[TRAFODION-2428] bash installer: always copy bashrc template file to 
trafodion's $HOME




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #901: [TRAFODION-2291] Ambari integration

[mkby]

Github user mkby commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/901#discussion_r94736506
  
--- Diff: install/README.md ---
@@ -0,0 +1,75 @@
+
+
+## Trafodion Installers
+
+* **install** - This is the current command-line installer. It installs a 
server tarball
+  on an existing CDH or HDP Hadoop cluster.
+* **python-installer** - This is the new command-line installer, meant to 
replace current
+  command-line installer. Likewise, installs server tarball on existing 
CDH, HDP,
+  or APACHE cluster.
+* **ambari-installer** - This integrates with Ambari cluster manager, so 
only applies to HDP.
+  In this case, trafodion server is installed via RPM. This is installed 
on Ambari server as
+  a management pack. Trafodion can be included in the initial cluster 
creation or added later.
+
+## Ambari Integration
+
+The Ambari MPack (management pack) is also packaged as an RPM, having a 
dependency on ambari-server.
+Given a proper yum repo file, `traf_ambari` rpm
+can be installed directly and it pulls in ambari-server.
+If ambari-server is previously installed and running, it must be restarted 
to pick up the Trafodion
+management pack.
+
+ Packaging
+
+Part of Ambari's job is to set up yum repo files on each node in order to 
install packages.
+The default URLs are for Hortonworks' public repos. But since your 
custom-built Trafodion is
+not hosted there, you need to specify a URL for your local yum repo 
server. To build that into
+the `traf_ambari` package, use make to specify value of `REPO_URL`.
+
+   `make package REPO_URL=http://my.repo.server/repo/...`
--- End diff --

What does this `REPO_URL` do? If we set an invalid repo url, does it still 
work or not?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #896: [TRAFODION-2410] Support install Traf...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/896

[TRAFODION-2410] Support install Trafodion on customized CDH parcel d…

…irectory

also changes option name from 'upgrade' to 'reinstall'.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2410

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/896.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #896


commit 9a888d2034748d4f9a60fb1d621f2c010c36ebb4
Author: Eason <hfutm...@gmail.com>
Date:   2016-12-26T08:40:55Z

[TRAFODION-2410] Support install Trafodion on customized CDH parcel 
directory




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #888: [TRAFODION-2417] Installer pre-work f...

[mkby]

Github user mkby commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/888#discussion_r93811870
  
--- Diff: core/sqf/sqenvcom.sh ---
@@ -139,6 +139,10 @@ else
 fi
 export TRAF_HOME=$PWD
 
+# normal installed location, can be overridden in .trafodion
+export DCS_INSTALL_DIR=$TRAF_HOME/dcs-$TRAFODION_VER
+export REST_INSTALL_DIR=$TRAF_HOME/rest-$TRAFODION_VER
+
--- End diff --

Thanks, looks good!


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #888: [TRAFODION-2417] Installer pre-work f...

[mkby]

Github user mkby commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/888#discussion_r93724802
  
--- Diff: core/sqf/sqenvcom.sh ---
@@ -139,6 +139,10 @@ else
 fi
 export TRAF_HOME=$PWD
 
+# normal installed location, can be overridden in .trafodion
+export DCS_INSTALL_DIR=$TRAF_HOME/dcs-$TRAFODION_VER
+export REST_INSTALL_DIR=$TRAF_HOME/rest-$TRAFODION_VER
+
--- End diff --

I like this idea, but where does `$TRAFODION_VER` variable come from?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #880: [TRAFODION-2393] python installer - r...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/880

[TRAFODION-2393] python installer - reorganize script directories for…

… better user experience

along with some code enhancements.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2393

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/880.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #880


commit b9a6795c1d80a2df9257c02e9a53d48a8af33315
Author: Eason <hfutm...@gmail.com>
Date:   2016-12-16T13:40:17Z

[TRAFODION-2393] python installer - reorganize script directories for 
better user experience




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #873: [TRAFODION-2391]: monitor failed to s...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/873

[TRAFODION-2391]: monitor failed to start when hostname contains uppe…

…rcase.

return lowercase hostname in ‘sqnodestatus’ to match the hostname in 
monitor.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion 2391

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/873.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #873


commit 271f1846192d7d0931766c1481a736adc4f1dc53
Author: Eason <hfutm...@gmail.com>
Date:   2016-12-08T02:22:54Z

[TRAFODION-2391]: monitor failed to start when hostname contains uppercase.




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #862: [TRAFODION-2384] Trafodion node needs...

[mkby]

Github user mkby commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/862#discussion_r90759767
  
--- Diff: install/python-installer/traf_setup.py ---
@@ -50,6 +50,9 @@ def run():
 ### kernel settings ###
 run_cmd('sysctl -w kernel.pid_max=65535 2>&1 > /dev/null')
 run_cmd('echo "kernel.pid_max=65535" >> /etc/sysctl.conf')
+run_cmd('cp $SQ_ROOT/sysinstall/etc/init.d/trafodion /etc/init.d')
--- End diff --

$SQ_ROOT should be SQ_ROOT, it's a local variable in this script.
`run_cmd('cp %s/sysinstall/etc/init.d/trafodion /etc/init.d' % SQ_ROOT)`


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #807: [TRAFODION-1839] Trafodion Installer ...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/807

[TRAFODION-1839] Trafodion Installer Evolution



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/807.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #807


commit 904b53df5c75f0dd5b0bd84e483c7d060023a7d6
Author: Eason <hfutm...@gmail.com>
Date:   2016-11-01T08:53:27Z

[TRAFODION-1839] Trafodion Installer Evolution




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #758: [TRAFODION-2278] rmscheck should use ...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/758

[TRAFODION-2278] rmscheck should use relative path

rmscheck should use relative path of sql file location.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/758.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #758


commit 56e498ff081b70d967f83f5379c197ef2d3db3b1
Author: Eason <hfutm...@gmail.com>
Date:   2016-10-12T03:11:41Z

[TRAFODION-2278] rmscheck should use relative path

rmscheck should use relative path of sql file location.




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #670: [TRAFODION-2183] remove warning in ck...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/670

[TRAFODION-2183] remove warning in ckillall

ckillall should remove the confused warning message

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/670.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #670


commit 5743270c88b3fc0db7fad81a411f7a720eb9af43
Author: Eason <hfutm...@gmail.com>
Date:   2016-08-24T03:36:11Z

[TRAFODION-2183] remove warning in ckillall

ckillall should remove the confused warning message




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #619: [TRAFODION-2115]

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/619

[TRAFODION-2115]



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/619.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #619


commit b4fa69cd532270ed8d6088f501d0a3c4a3bb86fe
Author: Eason <hfutm...@gmail.com>
Date:   2016-07-25T14:07:18Z

Create README.md




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request #606: [TRAFODION 2115]

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/606

[TRAFODION 2115]

Better UX for README on GitHub main page.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/606.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #606


commit 926ce9fe11f2a578b9c5e1fa3c88abef0d5d543e
Author: Eason <hfutm...@gmail.com>
Date:   2016-07-19T07:05:35Z

[TRAFODION 2115] 

Better UX for README on GitHub main page.




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request: TRAFODION-1706

[mkby]

Github user mkby commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/418#discussion_r58813434
  
--- Diff: install/installer/traf_config_setup ---
@@ -407,57 +412,131 @@ else
 fi
 
 #==
-#Hadoop user name
-
-echo -n "Enter Hadoop admin username, default is [$ADMIN]: "
+#HADOOP_TYPE
+echo -n "Enter installed hadoop type: cloudera, hortonworks, vanilla, 
default is [$HADOOP_TYPE]: "
--- End diff --

suggest to ask user only Y or N , select Y if they want to install 
trafodion on vanilla hadoop


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request: TRAFODION-1706

[mkby]

Github user mkby commented on a diff in the pull request:

https://github.com/apache/incubator-trafodion/pull/367#discussion_r55384539
  
--- Diff: core/sqf/sqenvcom.sh ---
@@ -140,18 +139,41 @@ fi
 export MY_SQROOT=$PWD
 export SQ_HOME=$PWD
 
+# set common version to be consistent between shared lib and maven 
dependencies
+export HBASE_DEP_VER_CDH=1.0.0-cdh5.4.4
+export HIVE_DEP_VER_CDH=1.1.0-cdh5.4.4
+export HBASE_DEP_VER_HDP=1.1.2.2.3.2.0-2950
+export HIVE_DEP_VER_HDP=1.2.1.2.3.2.0-2950
+export HBASE_DEP_VER_APACHE=1.0.2
+export HIVE_DEP_VER_APACHE=1.1.0
+export THRIFT_DEP_VER=0.9.0
+export HBASE_TRX_ID_CDH=hbase-trx-cdh5_4
+export HBASE_TRX_ID_APACHE=hbase-trx-apache1_0_2
+export HBASE_TRX_ID_HDP=hbase-trx-hdp2_3
+
 # general Hadoop & TRX dependencies - not distro specific, choose one to 
build against
 export HBASE_TRXDIR=$MY_SQROOT/export/lib
-export HBASE_TRX_ID=hbase-trx-cdh5_3
-export HBASE_DEP_VER=0.98.6-cdh5.3.0
-export HBASE_TRX_JAR=${HBASE_TRX_ID}-${TRAFODION_VER}.jar
-export UTIL_JAR=trafodion-utility-${TRAFODION_VER}.jar
-if [[ "$SQ_HBASE_DISTRO" = "HDP" ]]; then
-export HBASE_TRX_JAR=hbase-trx-hdp2_2-${TRAFODION_VER}.jar
+export HBASE_TRX_JAR=${HBASE_TRX_ID_CDH}-${TRAFODION_VER}.jar
+if [[ "$HBASE_DISTRO" = "HDP" ]]; then
+export HBASE_TRX_JAR=${HBASE_TRX_ID_HDP}-${TRAFODION_VER}.jar
 fi
-# set common version to be consistent between shared lib and maven 
dependencies
-export THRIFT_DEP_VER=0.9.0
-export HIVE_DEP_VER=0.13.1
+if [[ "$HBASE_DISTRO" = "APACHE" ]]; then
+export HBASE_TRX_JAR=${HBASE_TRX_ID_APACHE}-${TRAFODION_VER}.jar
--- End diff --

If using Apache Hadoop, the script will detect the system as an Apache 
Hadoop environment, so it will jump to line 487, and check line 618, the script 
will ``export HBASE_TRX_JAR=hbase-trx-hbase_98_4-${TRAFODION_VER}.jar``, will 
this overwrite the ``HBASE_TRX_JAR`` you set here?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request: [TRAFODION-1860]sqgen failed to ...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/358

[TRAFODION-1860]sqgen failed to execute on CentOS7

remove dependency of ctime.pl

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/358.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #358


commit 4ad51c730a98d2fff01a84ad86516a33cdbf55f6
Author: Eason <hfutm...@gmail.com>
Date:   2016-03-03T03:55:58Z

[TRAFODION-1860]sqgen failed to execute on CentOS7

remove dependency of ctime.pl




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-trafodion pull request: TRAFODION-1451: Vanilla hadoop s...

[mkby]

GitHub user mkby opened a pull request:

https://github.com/apache/incubator-trafodion/pull/80

TRAFODION-1451: Vanilla hadoop support for installer

jira 1451

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mkby/incubator-trafodion master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-trafodion/pull/80.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #80


commit fad0c54ea5defde1d45b8b8fff303f354542c782
Author: Eason <hfutm...@gmail.com>
Date:   2015-09-14T16:09:07Z

TRAFODION-1451: Vanilla hadoop support for installer




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---