[GitHub] incubator-trafodion pull request: [TRAFODION-1526]Incorrect messag...

2015-10-14 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/121#discussion_r42009490 --- Diff: core/conn/jdbc_type4/src/org/trafodion/jdbc/t4/TrafT4ResultSet.java --- @@ -4299,12 +4299,10 @@ private BaseRow getCurrentRow()

[GitHub] incubator-trafodion pull request: [TRAFODION-1526]Incorrect messag...

2015-10-14 Thread DaveBirdsall
Github user DaveBirdsall commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/121#discussion_r42009368 --- Diff: core/conn/jdbc_type4/src/org/trafodion/jdbc/t4/TrafT4ResultSet.java --- @@ -4299,12 +4299,10 @@ private BaseRow getCurrentRow()

[GitHub] incubator-trafodion pull request: Fix for Phoenix T2 test failures...

2015-10-14 Thread zellerh
GitHub user zellerh opened a pull request: https://github.com/apache/incubator-trafodion/pull/123 Fix for Phoenix T2 test failures due to missing DTM classes. The new trafodion-dtm-1.2.0.jar file needed to be added to the Phoenix tests as well, those are using Maven, not the

[GitHub] incubator-trafodion pull request: [TRAFODION-1527] Dcs Webui shows...

2015-10-14 Thread hegdean
GitHub user hegdean opened a pull request: https://github.com/apache/incubator-trafodion/pull/122 [TRAFODION-1527] Dcs Webui shows the dcs version as unknown Added a new target to Makefile(toplevel and core) to install traf components (Ex: make trafinstall) Updated the

[GitHub] incubator-trafodion pull request: [TRAFODION-1525]Locale for JDBCT...

2015-10-14 Thread arvind-narain
Github user arvind-narain commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/120#discussion_r42021375 --- Diff: core/conn/jdbc_type4/src/org/trafodion/jdbc/t4/HPT4Messages.java --- @@ -295,7 +292,7 @@ static HPT4Exception

[GitHub] incubator-trafodion pull request: [TRAFODION-1526]Incorrect messag...

2015-10-14 Thread arvind-narain
Github user arvind-narain commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/121#discussion_r42017420 --- Diff: core/conn/jdbc_type4/src/org/trafodion/jdbc/t4/TrafT4ResultSet.java --- @@ -4299,12 +4299,10 @@ private BaseRow getCurrentRow()

[GitHub] incubator-trafodion pull request: [TRAFODION-1526]Incorrect messag...

2015-10-14 Thread kevinxu021
GitHub user kevinxu021 opened a pull request: https://github.com/apache/incubator-trafodion/pull/121 [TRAFODION-1526]Incorrect messages for ResultSet cursor before the first row and after the last row The fixes are trying to separate the message into 2 pieces(before first row and