[GitHub] trafodion pull request #1688: [TRAFODION-3034] Support Oracle Hierarchy Quer...

2018-10-05 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1688#discussion_r223166271 --- Diff: core/sql/optimizer/ValueDesc.cpp --- @@ -4555,7 +4555,7 @@ void ValueIdSet::unparse(NAString &result, NAString connector

[GitHub] trafodion pull request #1714: [TRAFODION-3038] fix HDP download link in inst...

2018-09-13 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1714 [TRAFODION-3038] fix HDP download link in install_local_hadoop script You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm/trafodion

[GitHub] trafodion pull request #1711: [TRAFODION-3201] revert wrong changes in TRAFO...

2018-09-09 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1711 [TRAFODION-3201] revert wrong changes in TRAFODION-3175 I had a wrong understanding of the outputInfo class, so a wrong 'fix' in TRAFODION-3175 it can save as many rows , the num

[GitHub] trafodion pull request #1710: [TRAFODION-3193]update cqd reference manual ab...

2018-09-08 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1710 [TRAFODION-3193]update cqd reference manual about the new CQD Add a new CQD which end user will use, so update the manual You can merge this pull request into a Git repository by running

[GitHub] trafodion pull request #1704: [TRAFODION-2952] large amount of data will cau...

2018-08-28 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1704#discussion_r213515995 --- Diff: core/sql/cli/Cli.cpp --- @@ -10599,7 +10599,10 @@ static Lng32 SeqGenCliInterfaceUpdAndValidateMulti( numTries

[GitHub] trafodion pull request #1704: [TRAFODION-2952] large amount of data will cau...

2018-08-28 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1704#discussion_r213515950 --- Diff: core/sql/sqlcomp/DefaultConstants.h --- @@ -2946,6 +2946,12 @@ enum DefaultConstants // this is used to change cache size of sequence

[GitHub] trafodion pull request #1704: [TRAFODION-2952] large amount of data will cau...

2018-08-25 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1704 [TRAFODION-2952] large amount of data will cause error in sequence ge… …nerating Currently, the sequence next range value is NOT protected by DTM, but a retry logic. If

[GitHub] trafodion pull request #1698: [TRAFODION-2968] update SQL manual for MySQL f...

2018-08-23 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1698#discussion_r212510010 --- Diff: docs/sql_reference/src/asciidoc/_chapters/sql_functions_and_expressions.adoc --- @@ -11290,6 +11295,99 @@ ID --- 1 row(s) selected

[GitHub] trafodion pull request #1698: [TRAFODION-2968] update SQL manual for MySQL f...

2018-08-23 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1698#discussion_r212510081 --- Diff: docs/sql_reference/src/asciidoc/_chapters/sql_functions_and_expressions.adoc --- @@ -442,7 +444,10 @@ the expressions have NULL values, the

[GitHub] trafodion pull request #1698: [TRAFODION-2968] update SQL manual for MySQL f...

2018-08-23 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1698#discussion_r212509865 --- Diff: docs/sql_reference/src/asciidoc/_chapters/sql_functions_and_expressions.adoc --- @@ -11290,6 +11295,99 @@ ID --- 1 row(s) selected

[GitHub] trafodion pull request #1698: [TRAFODION-2968] update SQL manual for MySQL f...

2018-08-23 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1698#discussion_r212509677 --- Diff: docs/sql_reference/src/asciidoc/_chapters/sql_functions_and_expressions.adoc --- @@ -442,7 +444,10 @@ the expressions have NULL values, the

[GitHub] trafodion pull request #1699: [TRAFODION-2404] enhance the regexpr performan...

2018-08-20 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1699#discussion_r211479574 --- Diff: core/sql/exp/exp_like.cpp --- @@ -513,11 +512,16 @@ ex_expr::exp_return_type ExRegexpClauseChar::eval(char *op_data[], str_cpy_all

[GitHub] trafodion pull request #1699: [TRAFODION-2404] enhance the regexpr performan...

2018-08-19 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1699 [TRAFODION-2404] enhance the regexpr performance Previously, each time evaluate a row, it needs to invoke the regcomp(), by this change, only when the pattern is changed, invoke the regcomp

[GitHub] trafodion pull request #1698: [TRAFODION-2968] update SQL manual for MySQL f...

2018-08-19 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1698 [TRAFODION-2968] update SQL manual for MySQL functions unix_timestamp, uuid, sleep etc add documentation for some newly added MySQL functions. For comparability enhancement of Trafodion. You

[GitHub] trafodion pull request #1691: [TRAFODION-3182] cleanup monitor log content

2018-08-12 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1691 [TRAFODION-3182] cleanup monitor log content At field, when we want to do trafodion trouble shooting. Open mon.xxx.log file, in many times, it is flood with message related STDIN redirection

[GitHub] trafodion pull request #1690: [TRAFODION-1881] A better way to solve TRAFODI...

2018-08-12 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1690 [TRAFODION-1881] A better way to solve TRAFODION-1858 In my previous fix to TRAFODION-1858, I add a new method to check if a column is a system column by comparing the column name with "_

[GitHub] trafodion pull request #1689: [TRAFODION-3175] enhance ExeCliInterface get i...

2018-08-11 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1689 [TRAFODION-3175] enhance ExeCliInterface get interface to return null indicator enhance the ExeCliInterface get method to return NULL ptr for NULL value. So the caller is able to determine

[GitHub] trafodion pull request #1688: [TRAFODION-3034] Support Oracle Hierarchy Quer...

2018-08-10 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1688#discussion_r209260745 --- Diff: core/sql/optimizer/BindRelExpr.cpp --- @@ -2505,7 +2505,62 @@ RelExpr *RelExpr::bindSelf(BindWA *bindWA) if (bindWA

[GitHub] trafodion pull request #1688: [TRAFODION-3034] Support Oracle Hierarchy Quer...

2018-08-10 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1688 [TRAFODION-3034] Support Oracle Hierarchy Query (Connect By) This PR is the first patch to support Oracle Hierarchy query feature (CONNECT BY). In this PR, the feature is implemented as a

[GitHub] trafodion pull request #1686: [TRAFODION-3172] update documentation for sys_...

2018-08-07 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1686 [TRAFODION-3172] update documentation for sys_guid function update the doc for new added function You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] trafodion pull request #1679: [TRAFODION-3163] Add script to do STFS health ...

2018-08-06 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1679#discussion_r208065409 --- Diff: core/sqf/src/tm/tools/dtmci.cpp --- @@ -1801,6 +1801,20 @@ int main(int argc, char *argv[]) << "' fo

[GitHub] trafodion pull request #1680: [TRAFODION-3173] enhance ExeCliInterface fetch...

2018-08-05 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1680 [TRAFODION-3173] enhance ExeCliInterface fetchAllRows to save the dat… …a type You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm

[GitHub] trafodion pull request #1679: [TRAFODION-3163] Add script to do STFS health ...

2018-08-05 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1679 [TRAFODION-3163] Add script to do STFS health check It is hard to test the STFS status if the cluster has many nodes and the STFS configured with many directories. This script will help to do

[GitHub] trafodion pull request #1676: TRAFODION-3165 HDFS storage option not work we...

2018-08-02 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1676 TRAFODION-3165 HDFS storage option not work well if table use namespace If the table has namespace, the path of that table in HDFS should not in /hbase/data/default, but in /hbase/data// You

[GitHub] trafodion pull request #1666: TRAFODION-3167 add Maven Central Repository UR...

2018-07-28 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1666 TRAFODION-3167 add Maven Central Repository URL for sql pom.xml You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm/trafodion TRAFODION

[GitHub] trafodion pull request #1661: fix regression failure

2018-07-24 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1661 fix regression failure Sorry, need yet another fix to the GET command display change, which causing failures. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] trafodion pull request #1659: [TRAFODION-3162] fix regression failure get co...

2018-07-23 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1659 [TRAFODION-3162] fix regression failure get command fix regression failure You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm/trafodion

[GitHub] trafodion pull request #1657: [TRAFODION-3147] support sys_guid function

2018-07-21 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1657 [TRAFODION-3147] support sys_guid function Support Oracle sys_guid function as described: https://docs.oracle.com/cd/B12037_01/server.101/b10759/functions153.htm This is required by

[GitHub] trafodion pull request #1646: [TRAFODION-3145] Make "hbase_options" as defau...

2018-07-16 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1646#discussion_r202908302 --- Diff: core/sql/sqlcomp/CmpSeabaseDDLcommon.cpp --- @@ -10991,7 +10997,20 @@ CmpSeabaseDDL::setupHbaseOptions(ElemDDLHbaseOptions * hbaseOptionsClause

[GitHub] trafodion pull request #1646: [TRAFODION-3145] Make "hbase_options" as defau...

2018-07-15 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1646 [TRAFODION-3145] Make "hbase_options" as default setting and use cqd … …to change the default setting As we know, for performance thinking, we always need to add below sy

[GitHub] trafodion pull request #1619: [TRAFODION-3101] enhance get command to show r...

2018-06-24 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1619#discussion_r197692791 --- Diff: core/sql/regress/compGeneral/EXPECTED023 --- @@ -23,6 +23,10 @@ SB_HISTOGRAMS SB_HISTOGRAM_INTERVALS SB_PERSISTENT_SAMPLES

[GitHub] trafodion pull request #1619: [TRAFODION-3101] enhance get command to show r...

2018-06-24 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1619#discussion_r197692662 --- Diff: core/sql/regress/compGeneral/EXPECTED023 --- @@ -23,6 +23,10 @@ SB_HISTOGRAMS SB_HISTOGRAM_INTERVALS SB_PERSISTENT_SAMPLES

[GitHub] trafodion pull request #1619: [TRAFODION-3101] enhance get command to show r...

2018-06-24 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1619#discussion_r197666711 --- Diff: core/sql/executor/ExExeUtilGet.cpp --- @@ -949,6 +949,7 @@ short ExExeUtilGetMetadataInfoTcb::displayHeading() { if (getMItdb

[GitHub] trafodion pull request #1619: [TRAFODION-3101] enhance get command to show r...

2018-06-24 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1619 [TRAFODION-3101] enhance get command to show rows get At field, engineers issue get tables to list all tables, and want to count how many tables got, it will be good to show that info by 

[GitHub] trafodion pull request #1613: TRAFODION-3111 CentOS 7 build error

2018-06-18 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1613 TRAFODION-3111 CentOS 7 build error Some new code checkin broke the build on CentOS 7 I will continously monitor the CentOS 7 build until the jenkins setup. CentOS 7 jenkins test now

[GitHub] trafodion pull request #1532: [TRAFODION-3026] add create option storage pol...

2018-06-07 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1532#discussion_r193922170 --- Diff: core/sql/src/main/java/org/trafodion/sql/HBaseClient.java --- @@ -538,7 +564,21 @@ public boolean createk(String tblName, Object[] tableOptions

[GitHub] trafodion pull request #1532: [TRAFODION-3026] add create option storage pol...

2018-06-07 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1532#discussion_r193921823 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/coprocessor/transactional/TrxRegionEndpoint.java.tmpl --- @@ -2404,6

[GitHub] trafodion pull request #1532: [TRAFODION-3026] add create option storage pol...

2018-06-07 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1532#discussion_r193921772 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java --- @@ -3226,5 +3260,65

[GitHub] trafodion pull request #1532: [TRAFODION-3026] add create option storage pol...

2018-06-07 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1532#discussion_r193921336 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java --- @@ -2996,6 +3022,11

[GitHub] trafodion pull request #1532: [TRAFODION-3026] add create option storage pol...

2018-06-07 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1532#discussion_r193921620 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/client/transactional/TransactionManager.java --- @@ -3226,5 +3260,65

[GitHub] trafodion pull request #1592: Trafodion 3025 support order by with limit N i...

2018-06-02 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1592 Trafodion 3025 support order by with limit N in subquery ORDER BY in subquery is ignored by Trafodion, when in the subquery, there is a limit N, it is good to keep the order by: select

[GitHub] trafodion pull request #1591: TRAFODION-3076 update pom.xml.hdp

2018-06-01 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1591 TRAFODION-3076 update pom.xml.hdp for protobuf compile You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm/trafodion TRAFODION-3076

[GitHub] trafodion pull request #1571: [TRAFODION-3075] DCS start fail at CentOS 7

2018-05-20 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1571 [TRAFODION-3075] DCS start fail at CentOS 7 This is a temp solution. Root cause need much more time to figure out. You can merge this pull request into a Git repository by running

[GitHub] trafodion pull request #1570: [TRAFODION-3015] query no result when comparin...

2018-05-19 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1570 [TRAFODION-3015] query no result when comparing string with BIGNUM When the column is BIGNUM, and the predicate is comparing to a string. Trafodion cast the string into FLOAT and the convert

[GitHub] trafodion pull request #1569: [TRAFODION-3075] Installer needs to support RH...

2018-05-18 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1569 [TRAFODION-3075] Installer needs to support RH7 / Centos7 You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm/trafodion TRAFODION-3075

[GitHub] trafodion pull request #1564: [TRAFODION-3015] retrieve a value from numeric...

2018-05-15 Thread traflm
Github user traflm closed the pull request at: https://github.com/apache/trafodion/pull/1564 ---

[GitHub] trafodion pull request #1564: [TRAFODION-3015] retrieve a value from numeric...

2018-05-13 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1564 [TRAFODION-3015] retrieve a value from numeric type get no result When the column is defined as NUMERIC(19,0), underlying data type will use FLOAT to save the number. So an implicit cast

[GitHub] trafodion pull request #1563: [TRAFODION-3015] retrieve a value from numeric...

2018-05-13 Thread traflm
Github user traflm closed the pull request at: https://github.com/apache/trafodion/pull/1563 ---

[GitHub] trafodion pull request #1563: [TRAFODION-3015] retrieve a value from numeric...

2018-05-12 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1563 [TRAFODION-3015] retrieve a value from numeric type get no result if… … using xx='value' When the column is defined as NUMERIC(19,0), underlying data type will use FLO

[GitHub] trafodion pull request #1553: [TRAFODION-1861] Run Trafodion on CentOS 7

2018-05-08 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1553#discussion_r186920564 --- Diff: core/sqf/src/seabed/src/Makefile --- @@ -55,6 +55,16 @@ LIBSBUTIL= $(LIBEXPDIR)/libsbutil.so LIBSQSTATESB = $(LIBEXPDIR

[GitHub] trafodion pull request #1553: [TRAFODION-1861] Run Trafodion on CentOS 7

2018-05-07 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1553 [TRAFODION-1861] Run Trafodion on CentOS 7 Solve the link issue when using GCC 4.8 on CentOS 7 So Trafodion can start on CentOS 7 with this change It also depends on https://github.com

[GitHub] trafodion pull request #1552: [TRAFODION-3061] eliminate building errors on ...

2018-05-07 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1552 [TRAFODION-3061] eliminate building errors on CentOS7 There are some coding cannot be compiled using GCC in CentOS 7. We need to continuously check and fix these issues until CentOS7 is

[GitHub] trafodion pull request #1550: [TRAFODION-3059] remove "mt_" column family

2018-05-06 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1550 [TRAFODION-3059] remove "mt_" column family All Trafodion tables have a second column family named "mt_", which was originally designed for SSCC. SSCC is not enabled now

[GitHub] trafodion pull request #1549: [TRAFODION-3054] update pom.xml.cdh for hbase-...

2018-05-05 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1549 [TRAFODION-3054] update pom.xml.cdh for hbase-trx to build protobuf The various pom.xml.cdh file can be used as pom.xml for maven input, to build protobuf java output for hbase-trx. As we

[GitHub] trafodion pull request #1540: [TRAFODION-2953] initial implementation of sma...

2018-04-25 Thread traflm
GitHub user traflm reopened a pull request: https://github.com/apache/trafodion/pull/1540 [TRAFODION-2953] initial implementation of small table in one Hbase Region This is the very first code drop for TRAFODION-2953, the major purpose is to get feedback to see if the design is OK

[GitHub] trafodion pull request #1540: [TRAFODION-2953] initial implementation of sma...

2018-04-25 Thread traflm
Github user traflm closed the pull request at: https://github.com/apache/trafodion/pull/1540 ---

[GitHub] trafodion pull request #1540: [TRAFODION-2953] initial implementation of sma...

2018-04-24 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1540#discussion_r183936345 --- Diff: core/sql/regress/tools/sbdefs --- @@ -22,7 +22,7 @@ -- defs included during seabase regr run #ifdef SEABASE_REGRESS -cqd

[GitHub] trafodion pull request #1540: [TRAFODION-2953] initial implementation of sma...

2018-04-24 Thread traflm
GitHub user traflm reopened a pull request: https://github.com/apache/trafodion/pull/1540 [TRAFODION-2953] initial implementation of small table in one Hbase Region This is the very first code drop for TRAFODION-2953, the major purpose is to get feedback to see if the design is OK

[GitHub] trafodion pull request #1540: [TRAFODION-2953] initial implementation of sma...

2018-04-24 Thread traflm
Github user traflm closed the pull request at: https://github.com/apache/trafodion/pull/1540 ---

[GitHub] trafodion pull request #1540: [TRAFODION-2953] initial implementation

2018-04-24 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1540 [TRAFODION-2953] initial implementation This is the very first code drop for TRAFODION-2953, the major purpose is to get feedback to see if the design is OK and shall we continue with this

[GitHub] trafodion pull request #1521: Do not merge

2018-04-22 Thread traflm
Github user traflm closed the pull request at: https://github.com/apache/trafodion/pull/1521 ---

[GitHub] trafodion pull request #1533: [TRAFODION-3037] install_local_hadoop script o...

2018-04-22 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1533 [TRAFODION-3037] install_local_hadoop script out of date (APACHE) On a brandnew develop machine, if one set the HBASE_DISTRO=APACHE, install_local_hadoop cannot correctly download required tar

[GitHub] trafodion pull request #1532: [TRAFODION-3026] add create option storage pol...

2018-04-21 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1532 [TRAFODION-3026] add create option storage policy Some user want Trafodion to support HSM provided by Hadoop 2.6.0. Tables can be stored on different type of storage hardware (SSD/HDD). Hadoop

[GitHub] trafodion pull request #1521: Do not merge

2018-04-16 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1521 Do not merge Just for test You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm/trafodion TRAFODION-2953 Alternatively you can review and

[GitHub] trafodion pull request #1518: [TRAFODION-3021] union, intersect, except shou...

2018-04-13 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1518 [TRAFODION-3021] union, intersect, except should be cacheable Make UNION, INTERSECT, EXCEPT cacheable. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] trafodion pull request #1510: [TRAFODION-3020] regression issue

2018-04-09 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1510 [TRAFODION-3020] regression issue one line of unused code in parser You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm/trafodion

[GitHub] trafodion pull request #1485: [TRAFODION-2999] make UUID non reserved keywor...

2018-04-03 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1485#discussion_r178998606 --- Diff: core/sql/common/ComResWords.cpp --- @@ -395,8 +395,7 @@ const ComResWord ComResWords::resWords_[] = { ComResWord("USAGE",

[GitHub] trafodion pull request #1500: [TRAFODION-3014] merge R2.2 changes into mainb...

2018-03-27 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1500 [TRAFODION-3014] merge R2.2 changes into mainbranch This is an effort to merge R2.2 branch into main branch. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] trafodion pull request #1485: [TRAFODION-2999] make UUID non reserved keywor...

2018-03-26 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1485#discussion_r177260101 --- Diff: core/sql/common/ComResWords.cpp --- @@ -395,8 +395,6 @@ const ComResWord ComResWords::resWords_[] = { ComResWord("USAGE",

[GitHub] trafodion pull request #1497: [TRAFODION-2986] minor issues in adoc found du...

2018-03-25 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1497 [TRAFODION-2986] minor issues in adoc found during mvn site During the release 2.2, trying to update the documentation, found several minor typos and syntax errors Doc fixing. You can merge

[GitHub] trafodion pull request #1481: [Trafodion-2335] support functions in the colu...

2018-03-22 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1481#discussion_r176618285 --- Diff: core/sql/regress/seabase/EXPECTED020 --- @@ -3945,4 +3945,35 @@ ?2 --- 1 row(s) selected

[GitHub] trafodion pull request #1491: [TRAFODION-3006] unix_timestamp runtime core d...

2018-03-21 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1491 [TRAFODION-3006] unix_timestamp runtime core dump issue Wrong input param for strptime, null pointer You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] trafodion pull request #1481: [Trafodion-2335] support functions in the colu...

2018-03-21 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1481#discussion_r176278679 --- Diff: core/sql/common/ComSmallDefs.h --- @@ -609,47 +609,59 @@ enum ComColumnClass { COM_UNKNOWN_CLASS #define COM_ALTERED_USER_COLUMN_LIT

[GitHub] trafodion pull request #1481: [Trafodion-2335] support functions in the colu...

2018-03-20 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1481#discussion_r175966341 --- Diff: core/sql/regress/seabase/EXPECTED020 --- @@ -3945,4 +3945,35 @@ ?2 --- 1 row(s) selected

[GitHub] trafodion pull request #1481: [Trafodion-2335] support functions in the colu...

2018-03-20 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1481#discussion_r175964689 --- Diff: core/sql/common/ComSmallDefs.h --- @@ -609,47 +609,59 @@ enum ComColumnClass { COM_UNKNOWN_CLASS #define COM_ALTERED_USER_COLUMN_LIT

[GitHub] trafodion pull request #1481: [Trafodion-2335] support functions in the colu...

2018-03-19 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1481#discussion_r175651051 --- Diff: core/sql/parser/ElemDDLColDef.h --- @@ -309,6 +312,8 @@ class ElemDDLColDef : public ElemDDLNode NABoolean seabaseSerialized_

[GitHub] trafodion pull request #1485: [TRAFODION-2999] make UUID non reserved keywor...

2018-03-19 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1485 [TRAFODION-2999] make UUID non reserved keyword You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm/trafodion TRAFODION-2999

[GitHub] trafodion pull request #1481: [Trafodion-2335] support functions in the colu...

2018-03-19 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1481#discussion_r175641075 --- Diff: core/sql/parser/sqlparser.y --- @@ -8734,6 +8738,30 @@ datetime_value_function : TOK_CURDATE '(' ')'

[GitHub] trafodion pull request #1483: [TRAFODION-2998] sleep execution code cannot b...

2018-03-19 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1483 [TRAFODION-2998] sleep execution code cannot build on CentOS 7 Wrong implementation in the sleep runtime code. And made it not able to be built on CentOS 7 because of bad data type conversion

[GitHub] trafodion pull request #1482: [TRAFODION-2998] sleep execution code cannot b...

2018-03-19 Thread traflm
Github user traflm closed the pull request at: https://github.com/apache/trafodion/pull/1482 ---

[GitHub] trafodion pull request #1482: [TRAFODION-2998] sleep execution code cannot b...

2018-03-19 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1482 [TRAFODION-2998] sleep execution code cannot build on CentOS 7 Wrong data type conversion , and wrong usage of labs You can merge this pull request into a Git repository by running: $ git

[GitHub] trafodion pull request #1481: [Trafodion-2335] support functions in the colu...

2018-03-18 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1481 [Trafodion-2335] support functions in the column default definition This PR add the support in column default definition the expansion to use following functions: uuid unix_timestamp

[GitHub] trafodion pull request #1471: [TRAFODION-2985] fix wrong download link

2018-03-12 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1471 [TRAFODION-2985] fix wrong download link Some download links are wrong, fix them. This time, try each link by hand one by one. Should be all correct... You can merge this pull request into a

[GitHub] trafodion pull request #1467: [TRAFODION-2985] Update Project Web Site for A...

2018-03-10 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1467#discussion_r173617726 --- Diff: docs/src/site/markdown/download.md --- @@ -37,34 +72,34 @@ To build Trafodion from source code, see the [Trafodion Contributor Guide](https

[GitHub] trafodion pull request #1467: [TRAFODION-2985] Update Project Web Site for A...

2018-03-09 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1467 [TRAFODION-2985] Update Project Web Site for Apache Trafodion Release… … 2.2.0 You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm

[GitHub] trafodion pull request #1444: [TRAFODION-2954] add MySQL function unix_times...

2018-02-21 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1444#discussion_r169868459 --- Diff: core/sql/regress/executor/TEST002 --- @@ -1188,6 +1188,38 @@ select * from regexp_test where c1 regexp '^(25[0-5]|2[0-4][0-9]|[0-1]{1}[0

[GitHub] trafodion pull request #1444: [TRAFODION-2954] add MySQL function unix_times...

2018-02-20 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1444#discussion_r169511829 --- Diff: core/sql/optimizer/GroupAttr.cpp --- @@ -1793,6 +1793,8 @@ void GroupAttributes::resolveCharacteristicInputs(const ValueIdSet& external

[GitHub] trafodion pull request #1444: [TRAFODION-2157] add MySQL function unix_times...

2018-02-15 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1444#discussion_r168649121 --- Diff: core/sql/exp/exp_function.cpp --- @@ -2501,6 +2518,100 @@ ex_expr::exp_return_type ExFunctionReverseStr::eval(char *op_data[], return

[GitHub] trafodion pull request #1444: [TRAFODION-2157] add MySQL function unix_times...

2018-02-14 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1444 [TRAFODION-2157] add MySQL function unix_timestamp,uuid,sleep This is to add three MySQL compatible functions into Trafodion. It is required for another JIRA to add unix_timestamp, uuid as

[GitHub] trafodion pull request #1441: [TRAFODION-2908] create table with wrong char ...

2018-02-06 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1441#discussion_r166279557 --- Diff: core/sql/bin/SqlciErrors.txt --- @@ -140,7 +140,7 @@ 1138 Z 9 ADVANCED CRTCL DIALOUT --- unused --- 1139 Z 9 BEGINNER

[GitHub] trafodion pull request #1441: [TRAFODION-2908] create table with wrong char ...

2018-02-05 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1441 [TRAFODION-2908] create table with wrong char length hang and crash add a check if the key length is too long report error. Otherwise, SQL will go into a very long process to build the encoded

[GitHub] trafodion pull request #1438: [TRAFODION-2951] group_concat on utf8 column w...

2018-02-03 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1438 [TRAFODION-2951] group_concat on utf8 column with chinese character get messy code the pivot synthesize type function not consider the charset of output. Add code to copy the charset from

[GitHub] trafodion pull request #1435: Trafodion 2942

2018-02-01 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1435 Trafodion 2942 update the NOTICE file year This is for Release R2.2 change. You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm

[GitHub] trafodion pull request #1406: [TRAFODION-2918] fix regression tests

2018-01-18 Thread traflm
GitHub user traflm opened a pull request: https://github.com/apache/trafodion/pull/1406 [TRAFODION-2918] fix regression tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/traflm/trafodion TRAFODION-2918 Alternatively you can

[GitHub] trafodion pull request #1399: [TRAFODION-2909] Add ROLLUP Function for *Traf...

2018-01-17 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1399#discussion_r162052172 --- Diff: docs/sql_reference/src/asciidoc/_chapters/sql_functions_and_expressions.adoc --- @@ -6337,6 +6337,300 @@ UPDATE persnl.job SET jobdesc

[GitHub] trafodion pull request #1399: [TRAFODION-2909] Add ROLLUP Function for *Traf...

2018-01-17 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1399#discussion_r162050572 --- Diff: docs/sql_reference/src/asciidoc/_chapters/sql_functions_and_expressions.adoc --- @@ -6337,6 +6337,300 @@ UPDATE persnl.job SET jobdesc

[GitHub] trafodion pull request #1394: [TRAFODION-2891] fix the bufoverrun Critical e...

2018-01-16 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1394#discussion_r161806802 --- Diff: core/sqf/src/seatrans/tm/hbasetmlib2/hbasetm.h --- @@ -139,9 +139,9 @@ class CHbaseTM : public JavaObjectInterfaceTM JM_REGTRUNCABORT

[GitHub] trafodion pull request #1394: [TRAFODION-2891] fix the bufoverrun Critical e...

2018-01-16 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1394#discussion_r161806495 --- Diff: core/conn/odbc/src/odbc/nsksrvrcore/srvrothers.cpp --- @@ -5153,7 +5153,7 @@ odbc_SQLSvc_GetSQLCatalogs_sme_

[GitHub] trafodion pull request #1394: [TRAFODION-2891] fix the bufoverrun Critical e...

2018-01-16 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1394#discussion_r161806194 --- Diff: core/conn/odbc/src/odbc/Common/linux/sqmem.cpp --- @@ -400,7 +400,7 @@ void TestPool(void * membase, int length) long j, index

[GitHub] trafodion pull request #1394: [TRAFODION-2891] fix the bufoverrun Critical e...

2018-01-16 Thread traflm
Github user traflm commented on a diff in the pull request: https://github.com/apache/trafodion/pull/1394#discussion_r161805913 --- Diff: core/conn/odb/src/odb.c --- @@ -5313,7 +5313,7 @@ static void etabadd(char type, char *run, int id

  1   2   >