[GitHub] incubator-trafodion pull request #1347: TRAFODION-2846 optimize for batchExc...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/1347 TRAFODION-2846 optimize for batchException message display You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion batchExceptionMsg Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/1347.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 #1347 commit 3fb277efbfccaf4f0b8439fe7b532baf98611085 Author: aven <shengchen...@esgyn.cn> Date: 2017-12-15T05:29:35Z TRAFODION-2846 optimize for batchException message display ---
[GitHub] incubator-trafodion pull request #1344: TRAFODION-2844 add strategy to dcsse...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/1344 TRAFODION-2844 add strategy to dcsserver for restart moxsrvr when mxosrvr down ,dcsserver will restart it , and if mxosrvr down a lot of times in a period of time , there should reject the restart. if time between 6 times age(default setting) and this time are a very long time, dcsserver should allow the restart You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion mxosrvrRestart Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/1344.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 #1344 commit d41654fe70fd67740651da503e3c73e1b64e4168 Author: aven <shengchen...@esgyn.cn> Date: 2017-12-14T10:35:54Z TRAFODION-2844 add strategy to dcsserver for restart moxsrvr ---
[GitHub] incubator-trafodion pull request #1209: TRAFODION-2717 odbc give wrong data ...
Github user mashengchen commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1209#discussion_r134139755 --- Diff: dcs/src/main/java/org/trafodion/dcs/master/listener/ListenerWorker.java --- @@ -86,39 +86,43 @@ public void run() { DataEvent dataEvent; while(true) { -// Wait for data to become available -synchronized(queue) { -while(queue.isEmpty()) { -try { -queue.wait(); -} catch (InterruptedException e) { +try { +// Wait for data to become available +synchronized(queue) { +while(queue.isEmpty()) { +try { +queue.wait(); +} catch (InterruptedException e) { +} } +dataEvent = queue.remove(0); } -dataEvent = queue.remove(0); -} -SelectionKey key = dataEvent.key; -SocketChannel client = (SocketChannel) key.channel(); -Socket s = client.socket(); -ClientData clientData = (ClientData) key.attachment(); -ListenerService server = dataEvent.server; -dataEvent.key = null; -dataEvent.server = null; - -switch (clientData.hdr.getOperationId()){ -case ListenerConstants.DCS_MASTER_GETSRVRAVAILABLE: -clientData = requestGetObjectRef.processRequest(clientData, s); -break; -case ListenerConstants.DCS_MASTER_CANCELQUERY: -clientData = requestCancelQuery.processRequest(clientData, s); -break; -default: -clientData = requestUnknown.processRequest(clientData, s); -break; +SelectionKey key = dataEvent.key; +SocketChannel client = (SocketChannel) key.channel(); +Socket s = client.socket(); +ClientData clientData = (ClientData) key.attachment(); +ListenerService server = dataEvent.server; +dataEvent.key = null; +dataEvent.server = null; + +switch (clientData.hdr.getOperationId()){ +case ListenerConstants.DCS_MASTER_GETSRVRAVAILABLE: +clientData = requestGetObjectRef.processRequest(clientData, s); +break; +case ListenerConstants.DCS_MASTER_CANCELQUERY: +clientData = requestCancelQuery.processRequest(clientData, s); +break; +default: +clientData = requestUnknown.processRequest(clientData, s); +break; +} +// Return to sender +int requestReply = clientData.requestReply; +key.attach(clientData); +server.send(new PendingRequest(key, requestReply)); +} catch (Exception e){ +LOG.error("Unexpected Exception", e); --- End diff -- in this case if not catch Exception, there will throw the exception and lead to dcsmaster thread died, then the client side will never get connection, and once there catch Exception the cancelConnection will set to true, and then requestReply will be REQUST_CLOSE, and client side will get response for connect failure, this will not influence other client side --- 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 #1209: TRAFODION-2717 odb give wrong data a...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/1209 TRAFODION-2717 odb give wrong data and dcsmaster hang You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion dcs_try_catch Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/1209.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 #1209 commit 3b9cb971136e93a65ad9e8570a15bf8180dad273 Author: mashengchen <mashengc...@gmail.com> Date: 2017-08-16T03:25:22Z TRAFODION-2717 odb give wrong data and dcsmaster hang --- 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 #1180: [TRAFODION-2685] int type doesnot ch...
Github user mashengchen commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/1180#discussion_r128412940 --- Diff: core/conn/jdbcT4/src/main/java/org/trafodion/jdbc/t4/TrafT4ResultSet.java --- @@ -1372,7 +1372,7 @@ public long getLong(int columnIndex) throws SQLException { } } else { throw TrafT4Messages.createSQLException(connection_.props_, connection_.getLocale(), - "numeric_out_of_range", null); + "numeric_out_of_range", bd.longValue()); --- End diff -- agree ,i will change this --- 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 #1181: [TRAFODION-2684] tinyint boundary is...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/1181 [TRAFODION-2684] tinyint boundary issue You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion jdbc_tinyint_boundary Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/1181.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 #1181 commit f816b97e311998e00f11ad61c790a6b4270dba92 Author: mashengchen <mashengc...@gmail.com> Date: 2017-07-14T09:18:14Z fix tinyint boundary issue --- 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 #1180: [TRAFODION-2685] int type doesnot ch...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/1180 [TRAFODION-2685] int type doesnot check boundary You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion jdbc_int_boundary Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/1180.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 #1180 commit 5370426f82acba0ebd748a5e41797bf2c57dc53f Author: mashengchen <mashengc...@gmail.com> Date: 2017-07-14T09:10:14Z fix int type doesnot check boundary --- 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 #1144: TRAFODION-2668 one mxosrvr cant star...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/1144 TRAFODION-2668 one mxosrvr cant startup, dcsServer down You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion mxo_startup_err Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/1144.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 #1144 commit baaaf8c86057dd350f7438d1f096151c83ee48da Author: mashengchen <mashengc...@gmail.com> Date: 2017-06-23T10:41:32Z TRAFODION-2668 one mxosrvr cant startup, dcsServer 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 #1124: TRAFODION-2385 bug fix
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/1124 TRAFODION-2385 bug fix there exists bug in #1098 , this PR is to fix it. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion jdbc_type_btl Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/1124.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 #1124 commit e24747269e0c6e9318e9c328957dde8c78bb41f9 Author: mashengchen <mashengc...@gmail.com> Date: 2017-06-19T02:42:35Z TRAFODION-2385 bug fix --- 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 #1098: TRAFODION-2385 jdbcT4 support boolea...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/1098 TRAFODION-2385 jdbcT4 support boolean, tinyint, largeint You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion jdbc_type_btl Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/1098.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 #1098 commit e32f6b9b3bff11c7609f6a2f8676e387ccefcfa7 Author: mashengchen <mashengc...@gmail.com> Date: 2017-05-18T10:29:56Z TRAFODION-2385 jdbcT4 support boolean, tinyint, largeint --- 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 #1088: TRAFODION-2385 jdbcT4 support boolea...
Github user mashengchen closed the pull request at: https://github.com/apache/incubator-trafodion/pull/1088 --- 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 #1095: TRAFODION-2462 jline not work will
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/1095 TRAFODION-2462 jline not work will use -j yes or no to decide whether using jline yes means use jline no means not use jline default is not use jline You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion trafci_jline Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/1095.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 #1095 commit 5bba13319d03021106c8844fd86ebc2381f95eda Author: mashengchen <mashengc...@gmail.com> Date: 2017-05-14T05:37:44Z TRAFODION-2462 jline not work will --- 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 #1088: TRAFODION-2385 jdbcT4 support boolea...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/1088 TRAFODION-2385 jdbcT4 support boolean, tinyint, largeint You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion jdbc_type_btl Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/1088.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 #1088 commit 3e4b74cdd3965fe68affe1025cb7090513e64d27 Author: mashengchen <mashengc...@gmail.com> Date: 2017-05-09T10:42:32Z TRAFODION-2385 jdbcT4 support boolean, tinyint, largeint --- 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 #897: TRAFODION-2421
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/897 TRAFODION-2421 You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion TRAFODION-2421 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/897.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 #897 commit 853314194f89c7af555c4cfabf658f62a9f98921 Author: mashengchen <mashengc...@gmail.com> Date: 2016-12-27T10:13:51Z TRAFODION-2421 --- 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 #886: TRAFODION-2402
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/886 TRAFODION-2402 replace TRAFCI_CLASSPATH in trafci.sh, previous is absolute path, now is ${TRAF_HOME}/traf/lib/xx.jar You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion 2402 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/886.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 #886 commit 76e9e87bb95c7a1fdca1cd4a4e0925b65b1344d0 Author: mashengchen <mashengc...@gmail.com> Date: 2016-12-22T03:12:00Z TRAFODION-2402 --- 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 #879: [TRAFODION-2211] trafci can use histo...
Github user mashengchen commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/879#discussion_r92960037 --- Diff: core/conn/trafci/src/main/java/org/trafodion/ci/ConsoleReader.java --- @@ -172,7 +176,8 @@ public String readLine() throws UnsupportedEncodingException, UserInterruption, --- End diff -- ohh, i really not notice that.. i will research how to add this behavior. --- 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 #879: [TRAFODION-2211] trafci can use histo...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/879 [TRAFODION-2211] trafci can use history cmd trafci should support up, down, left, right key in the keyboard You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion trafci_his Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/879.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 #879 commit 8618a51203e99a5075f3a3974d2c06b1ebdf1b60 Author: mashengchen <mashengc...@gmail.com> Date: 2016-12-13T08:19:15Z trafci can use history cmd --- 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 #857: TRAFODION-2374 insert data through jd...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/857 TRAFODION-2374 insert data through jdbc while column type is numeric will insert error data insert negative numeric (-1) through jdbc addbatch while the column type is numeric(20,0). eg : prepStmt.setBigDecimal(1, new BigDecimal(-1)); the result is a strange number. seemingly a overflow You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion 2374 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/857.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 #857 commit e4838ec1e589a09fdedaa5445729e9f0110162bc Author: mashengchen <mashengc...@gmail.com> Date: 2016-11-24T08:24:17Z Merge pull request #1 from apache/master update commit 1daecf1318d164261104a5bce51ae5a4bf2dc3ad Author: mashengchen <mashengc...@gmail.com> Date: 2016-11-29T09:28:19Z TRAFODION-2374 --- 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 #810: [TRAFODION-2293] remove minor version...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/810 [TRAFODION-2293] remove minor version for the hbase-trx-apache_*.jar You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion trafodion-2293 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/810.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 #810 commit 8c38f482b8085f6f9676a0d6d93834e65e17795a Author: mashengchen <mashengc...@gmail.com> Date: 2016-11-02T02:58:21Z Update install_local_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 #784: [TRAFODION-2293] add restrict in inst...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/784 [TRAFODION-2293] add restrict in install_local_hadoop You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion trafodion-2293 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/784.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 #784 commit 961e0afbc73df1eb8c97f517ad3019943c6a11cd Author: mashengchen <mashengc...@gmail.com> Date: 2016-10-26T04:20:56Z [TRAFODION-2293] add restrict in install_local_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 #702: optimize using of udf
Github user mashengchen closed the pull request at: https://github.com/apache/incubator-trafodion/pull/702 --- 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 #702: optimize using of udf
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/702 optimize using of udf this is an enhancement of udf. user who want to use udf can extends SimpleUDR , and just archieve execute(TupleInfo tuple). the details are showed in SimpleUDRTest You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/702.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 #702 commit 97f63ab60e3ae07c32e673e90ac1285b37bd9cc1 Author: mashengchen <mashengc...@gmail.com> Date: 2016-09-14T06:14:27Z optimize using of tmudf --- 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 #671: [TRAFODION-2118] dynamic add coproces...
Github user mashengchen closed the pull request at: https://github.com/apache/incubator-trafodion/pull/671 --- 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 #671: [TRAFODION-2118] dynamic add coproces...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/671 [TRAFODION-2118] dynamic add coprocessor using Hbase api 1. mv coprocessor from org.apache.hbase.coprocessor.transactional to org.apache.trafodion.coprocessor, beacuse org.apache.hbase.coprocessor is build-in namespace and the coprocessors under this namespace will be block. 2. hbase-trx-*.jar will never put under $HBASE_HOME/lib , but put into HDFS which is under /hbase/lib. this will help to dynamic add coprocessor. 3. no need to change hbase-site.xml to add hbase.coprocessor.region.classes in the future 4. change related install scripts. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion 2118new Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/671.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 #671 commit 77e59d57f5563b3baaed7f71f95945e45569cce2 Author: mashengchen <mashengc...@gmail.com> Date: 2016-08-24T06:41:52Z [TRAFODION-2118] dynamic add coprocessor using Hbase api --- 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 #655: TRAFODION-2067 support cdh5.7 & hbase...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/655 TRAFODION-2067 support cdh5.7 & hbase1.2 previous PR #610 has some conflict. do a new PR 1. support cdh5.7 & hbase1.2. add hbase-trx-cdh5.7.jar & hbase-trx-apache1.2.jar 2. HBASE_DISTRO add CDH5.7 APACHE1.2 3. remove cdh & apache & cdh version in trafodion-dtm-.jar & trafodion-sql-.jar & trafodion-utility-.jar, they will be trafodion-dtm-[apache][cdh][hdp]-$trafodion_version.jar & trafodion-sql-[apache][cdh][hdp]-$trafodion_version.jar & trafodion-utility-$trafodion_version.jar . becacuse we can't determine which version user will install. and we only have one version of each jar. but not hbase-trx.jar, so hbase-trx*.jar have version, because they have multi version. eg: for cdh they have hbase-trx-cdh5_4-2.1.0.jar & hbase-trx-cdh5_5-2.1.0.jar & hbase-trx-cdh5_7-2.1.0.jar 4. HBASE1.* ->APACHE1.* in .tmpl files. 5. do some optimize in traf_config_check You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion cdh5.7 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/655.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 #655 commit 7329e015b62c13ee62d80240ace1766519de7c19 Author: mashengchen <mashengc...@gmail.com> Date: 2016-08-12T03:06:42Z TRAFODION-2067 support cdh5.7 & hbase1.2 commit e1799e431465d108a226ce726b1d21b28a3eaf9d Author: mashengchen <mashengc...@gmail.com> Date: 2016-08-12T03:41:55Z update SsccRegionEndpoint.java.tmpl --- 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 #610: TRAFODION-2067 support cdh5.7 & hbase...
Github user mashengchen closed the pull request at: https://github.com/apache/incubator-trafodion/pull/610 --- 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 #610: TRAFODION-2067 support cdh5.7 & hbase...
Github user mashengchen commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/610#discussion_r73470868 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/coprocessor/transactional/TrxRegionEndpoint.java.tmpl --- @@ -4271,8 +4291,12 @@ CoprocessorService, Coprocessor { } try { final WALKey wk = new WALKey(this.regionInfo.getEncodedNameAsBytes(), this.regionInfo.getTable(), EnvironmentEdgeManager.currentTime()); +#ifdef CDH5.7 APACHE1.2 +txid = this.tHLog.append(this.m_Region.getTableDesc(), this.regionInfo, wk, e, false); --- End diff -- no side effects --- 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 #610: TRAFODION-2067 support cdh5.7 & hbase...
Github user mashengchen commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/610#discussion_r73469972 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/coprocessor/transactional/TrxRegionEndpoint.java.tmpl --- @@ -4241,8 +4257,12 @@ CoprocessorService, Coprocessor { e.add(kv); try { final WALKey wk = new WALKey(this.regionInfo.getEncodedNameAsBytes(), this.regionInfo.getTable(), EnvironmentEdgeManager.currentTime()); +#ifdef CDH5.7 APACHE1.2 +txid = this.tHLog.append(this.m_Region.getTableDesc(), this.regionInfo, wk, e, false); --- End diff -- i do a test and find out there is no side effect before cdh5.7 - 2016-08-04 05:44:09,318 TRACE [B.defaultRpcServer.handler=17,queue=2,port=35946] transactional.TrxRegionEndpoint: TrxRegionEndpoint coprocessor: commit - txId 3, Write commit HLOG seq 274 do sqstop && swsopall and do sqstart && swstartall 2016-08-04 06:44:48,680 TRACE [B.defaultRpcServer.handler=29,queue=2,port=43771] transactional.TrxRegionEndpoint: TrxRegionEndpoint coprocessor: commit - txId 3, Write commit HLOG seq 214 cdh 5.7 --- 2016-08-04 06:18:52,101 TRACE [PriorityRpcServer.handler=5,queue=1,port=58401] transactional.TrxRegionEndpoint: TrxRegionEndpoint coprocessor: commit - txId 3, Write commit HLOG seq 227 --- 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 #610: TRAFODION-2067 support cdh5.7 & hbase...
Github user mashengchen commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/610#discussion_r73286115 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/coprocessor/transactional/SsccRegionEndpoint.java.tmpl --- @@ -236,7 +236,11 @@ CoprocessorService, Coprocessor { private HRegionInfo regionInfo = null; private HRegion m_Region = null; +#ifdef CDH5.7 APACHE1.2 + private HRegion t_Region = null; --- End diff -- the qualify of HRegion.getScanner(final Scan scan, final List scanners) has changed to public in hbase1.2, what the TransactionalRegion do before hbase1.2 is to make getScanner to be public , so in hbase1.2 it can change back to HRegion. and it may help to remove "hbase.hregion.impl" in hbase-site.xml in the future. --- 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 #632: TRAFODION-1743 add connection session
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/632 TRAFODION-1743 add connection session this jira has been merged in #561, but it is strange the changed code in apache/master disappeared. so i do a PR again. add seesion to each connection. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion jira1743 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/632.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 #632 commit b25f9437e3beb356959eaa000f1bab13c0b4a0f5 Author: mashengchen <mashengc...@gmail.com> Date: 2016-08-01T07:19:14Z trafodion-1743 add connection session --- 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 #610: TRAFODION-2067 support cdh5.7 & hbase...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/610 TRAFODION-2067 support cdh5.7 & hbase1.2 1. support cdh5.7 & hbase1.2 2. HBASE_DISTRO add CDH5.7 APACHE1.2 3. remove cdh & apache & cdh version in trafodion-dtm-*.jar & trafodion-sql-*.jar & trafodion-utility-*.jar, they will be trafodion-dtm-[apache][cdh][hdp]-$trafodion_version.jar & trafodion-sql-[apache][cdh][hdp]-$trafodion_version.jar & trafodion-utility-$trafodion_version.jar . becacuse we can't determine which version user will install. and we only have one version of each jar. but not hbase-trx*.jar, so hbase-trx*.jar have version, because they have multi version. eg: for cdh they have hbase-trx-cdh5_4-2.1.0.jar & hbase-trx-cdh5_5-2.1.0.jar & hbase-trx-cdh5_7-2.1.0.jar 4. HBASE1.* ->APACHE1.* in .tmpl files. 5. do some optimize in traf_config_check You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion cdh5.7 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/610.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 #610 commit a6b9b2d23b543f95b90102f6097d9087e508cb63 Author: mashengchen <mashengc...@gmail.com> Date: 2016-07-20T03:25:31Z Merge pull request #5 from apache/master update head commit ad9258c7fdedd4ef75ded4110f3702d3577fa232 Author: mashengchen <mashengc...@gmail.com> Date: 2016-07-20T04:07:23Z TRAFODION-2067 support CDH5.7 commit f523af4c07aa8528bfb88ed963a85a1e46caac76 Author: mashengchen <mashengc...@gmail.com> Date: 2016-07-21T02:35:44Z change some bug --- 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 #600: TRAFODION-2016 support hbase1.1 & cdh...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/600 TRAFODION-2016 support hbase1.1 & cdh5.5 support hbase 1.1 & cdh 5.5 for developer pls do set HBASE_DISTRO before source sqenv.sh export HBASE_DISTRO=APACHE1.0 (APACHE HBASE1.0) export HBASE_DISTRO=APACHE1.1 (APACHE HBASE1.1) export HBASE_DISTRO=CDH5.4 (cloudera 1.0.0-cdh5.4.4) export HBASE_DISTRO=CDH5.5 (cloudera 1.0.0-cdh5.5.1) export HBASE_DISTRO=HDP (hortonworks 2.3) HBASE_DISTRO will be determined by HADOOP_TYPE VERSION You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion hbase1.1 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/600.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 #600 commit b1ff89f145063ef633c8852495db875668c2fbd2 Author: traf <traf@aven01.novalocal> Date: 2016-06-27T05:21:01Z jira1743 commit 324a981c69e9c150bd7730d63ad5049483749918 Author: mashengchen <mashengc...@gmail.com> Date: 2016-06-29T07:42:59Z Merge pull request #1 from apache/master update head commit 6d08029a1cbd162b5573e3055ecfee6b45f8371a Author: traf <traf@aven01.novalocal> Date: 2016-06-30T09:48:59Z TRAFODION-2016 support apache hbase1.1 & cdh5.5 commit 443b0c4abfcb8a4493deecb805b76568d8d1ed74 Author: traf <traf@aven01.novalocal> Date: 2016-06-30T10:08:32Z remove other jira changes commit bacd8bff71ca271d1f8be92c03940bb2fbd4e575 Author: traf <traf@aven01.novalocal> Date: 2016-06-30T10:10:34Z remove other jira changes commit a6942309b0efba4f548a8e065e729826c0f8cca4 Author: mashengchen <mashengc...@gmail.com> Date: 2016-07-12T02:19:32Z Update sqenvcom.sh correct HBASE_DISTRO value for apache situation commit a0fd11ec72c32ab7a3817ebd71dd21aafb996410 Author: mashengchen <mashengc...@gmail.com> Date: 2016-07-12T02:24:48Z Update sqenvcom.sh sorry,it's a long time from my first commit, and i mistake the value for hbase_distro when in apache situation, now returne back, commit 0a5a0b8778a3d468345dac57a805f319cc9249b1 Author: mashengchen <mashengc...@gmail.com> Date: 2016-07-18T11:00:03Z Update traf_config_check add hbase_distro to trafodion_config --- 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 #568: TRAFODION-2016
Github user mashengchen closed the pull request at: https://github.com/apache/incubator-trafodion/pull/568 --- 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 #568: TRAFODION-2016
GitHub user mashengchen reopened a pull request: https://github.com/apache/incubator-trafodion/pull/568 TRAFODION-2016 support hbase 1.1 & cdh 5.5 for developer pls do set HBASE_DISTRO before source sqenv.sh export HBASE_DISTRO=APACHE1.0 (APACHE HBASE1.0) export HBASE_DISTRO=APACHE1.1 (APACHE HBASE1.1) export HBASE_DISTRO=CDH5.4(cloudera 1.0.0-cdh5.4.4) export HBASE_DISTRO=CDH5.5(cloudera 1.0.0-cdh5.5.1) export HBASE_DISTRO=HDP (hortonworks 2.3) You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion hbase1.1 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/568.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 #568 commit b1ff89f145063ef633c8852495db875668c2fbd2 Author: traf <traf@aven01.novalocal> Date: 2016-06-27T05:21:01Z jira1743 commit 324a981c69e9c150bd7730d63ad5049483749918 Author: mashengchen <mashengc...@gmail.com> Date: 2016-06-29T07:42:59Z Merge pull request #1 from apache/master update head commit 6d08029a1cbd162b5573e3055ecfee6b45f8371a Author: traf <traf@aven01.novalocal> Date: 2016-06-30T09:48:59Z TRAFODION-2016 support apache hbase1.1 & cdh5.5 commit 443b0c4abfcb8a4493deecb805b76568d8d1ed74 Author: traf <traf@aven01.novalocal> Date: 2016-06-30T10:08:32Z remove other jira changes commit bacd8bff71ca271d1f8be92c03940bb2fbd4e575 Author: traf <traf@aven01.novalocal> Date: 2016-06-30T10:10:34Z remove other jira changes commit eba153afbf16583745996efd050d19319662fd29 Author: mashengchen <mashengc...@gmail.com> Date: 2016-06-30T10:26:14Z Merge pull request #2 from apache/master update commit 024e001d384fc343cc380a3dccdaa7abbc0757d4 Author: Liu Ming <ovis_p...@sina.com> Date: 2016-07-07T15:03:36Z [TRAFODION-2096] create all tpcds hive external tables from install_local_hadoop commit a6942309b0efba4f548a8e065e729826c0f8cca4 Author: mashengchen <mashengc...@gmail.com> Date: 2016-07-12T02:19:32Z Update sqenvcom.sh correct HBASE_DISTRO value for apache situation commit a0fd11ec72c32ab7a3817ebd71dd21aafb996410 Author: mashengchen <mashengc...@gmail.com> Date: 2016-07-12T02:24:48Z Update sqenvcom.sh sorry,it's a long time from my first commit, and i mistake the value for hbase_distro when in apache situation, now returne back, commit fc3bb5a7c744848bd1c81680b34d1fbbf31e225b Author: Liu Ming <ovis_p...@sina.com> Date: 2016-07-12T13:02:10Z update the checking table commit ea2087410bcf23ff2800be03ca0fc8592fa0bb48 Author: Sandhya Sundaresan <sand...@apache.org> Date: 2016-07-12T23:22:58Z Merge remote branch 'origin/pr/585/head' into merge_585 commit 249dfefac344fbb7db72d1131fae15a75c50058d Author: Sandhya Sundaresan <sand...@apache.org> Date: 2016-07-14T00:18:01Z Adding AQR retry for ENOENT and EAGAIN which may be encountered after an hdfsOpen on a file that has gone through a timestamp check. Squashed commit of the following: commit 52551b6461e0b761aeb37958ecd8673ad29bec89 Author: Sandhya Sundaresan <sand...@apache.org> Date: Tue Jul 12 20:27:33 2016 + Check for NULL pointer commit a7bb285fb7874b314acb6c74cde5e0b6911d6930 Author: Sandhya Sundaresan <sand...@apache.org> Date: Tue Jul 12 16:12:17 2016 + Rework to pass errno at the lowest layer as output param commit abb236336ee68765fcf9c534fbeae3fe6af5cdb6 Author: Sandhya Sundaresan <sand...@apache.org> Date: Mon Jul 11 19:22:04 2016 + Removed all retries and added AQR retry for missing hdfs file after timestamp check. commit 2530c4ebb8274bea8394067d2d2212b5bc3c97ef Author: Sandhya Sundaresan <sand...@apache.org> Date: Mon Jul 11 07:11:35 2016 + timeout to 20 commit 825bd5be2b0db4006bf82a0a9348338f7fc66236 Author: Sandhya Sundaresan <sand...@apache.org> Date: Mon Jul 11 03:59:11 2016 + increase sleep commit e471a4655ca8efe469e5dd8ab14748660fa5cdec Author: Sandhya Sundaresan <sand...@apache.org> Date: Sun Jul 10 19:00:03 2016 + Unconditionally retrying on open errors. commit b0402243a17094f7fec9899a6afc5c31c48cc38b Author: Sandhya Sundaresan <sand...@apache.org> Date: Sun Jul 10 03:31:01 2016 + Retrying for ENOENT too commit cbb2175282095edb73b30e79a9728f689b80c435 Author: Sandhya Sundaresan <sand...@apache.org> Date: Sat Jul 9 23:14:31 2016 + Rework and returning errno commit dd99c2
[GitHub] incubator-trafodion pull request #568: TRAFODION-2016
Github user mashengchen closed the pull request at: https://github.com/apache/incubator-trafodion/pull/568 --- 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 #568: TRAFODION-2016
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/568 TRAFODION-2016 support hbase 1.1 & cdh 5.5 for developer pls do set HBASE_DISTRO before source sqenv.sh export HBASE_DISTRO=APACHE1.0 (APACHE HBASE1.0) export HBASE_DISTRO=APACHE1.1 (APACHE HBASE1.1) export HBASE_DISTRO=CDH5.4(cloudera 1.0.0-cdh5.4.4) export HBASE_DISTRO=CDH5.5(cloudera 1.0.0-cdh5.5.1) export HBASE_DISTRO=HDP (hortonworks 2.3) You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion hbase1.1 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/568.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 #568 commit b1ff89f145063ef633c8852495db875668c2fbd2 Author: traf <traf@aven01.novalocal> Date: 2016-06-27T05:21:01Z jira1743 commit 324a981c69e9c150bd7730d63ad5049483749918 Author: mashengchen <mashengc...@gmail.com> Date: 2016-06-29T07:42:59Z Merge pull request #1 from apache/master update head commit 6d08029a1cbd162b5573e3055ecfee6b45f8371a Author: traf <traf@aven01.novalocal> Date: 2016-06-30T09:48:59Z TRAFODION-2016 support apache hbase1.1 & cdh5.5 commit 443b0c4abfcb8a4493deecb805b76568d8d1ed74 Author: traf <traf@aven01.novalocal> Date: 2016-06-30T10:08:32Z remove other jira changes commit bacd8bff71ca271d1f8be92c03940bb2fbd4e575 Author: traf <traf@aven01.novalocal> Date: 2016-06-30T10:10:34Z remove other jira changes --- 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 #561: jira1743
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/561 jira1743 add seesion to each connection. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/561.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 #561 commit b1ff89f145063ef633c8852495db875668c2fbd2 Author: traf <traf@aven01.novalocal> Date: 2016-06-27T05:21:01Z jira1743 --- 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 #503: TRAFODION-2016
Github user mashengchen closed the pull request at: https://github.com/apache/incubator-trafodion/pull/503 --- 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
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/418 TRAFODION-1706 now user can use "trafodion_install" to install trafodion who had installed vanilla apache hadoop and hbase. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/418.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 #418 commit 0dbf1d1bc8718990850b4cff6e5f955a213209ae Author: mashengchen <mashengc...@gmail.com> Date: 2016-04-05T03:03:48Z Trafodion building troubles commit c0b3136c6e29afc97d299cfc3407df7093a20060 Author: mashengchen <mashengc...@gmail.com> Date: 2016-04-05T03:07:20Z Merge branch 'master' of github.com:mashengchen/incubator-trafodion commit 897ea4b4013e55c691ffabb84fef804c7a623bf8 Author: mashengchen <mashengc...@gmail.com> Date: 2016-04-05T03:09:01Z Merge pull request #13 from apache/master update head commit 674f1cf54052fc5abba37eb3f65a4462c860f607 Author: root <root@aven01.novalocal> Date: 2016-04-06T09:52:14Z support vanilla apache commit c505fcea152db153d8197df22253a6914f21a4fd Author: mashengchen <mashengc...@gmail.com> Date: 2016-04-06T09:58:08Z Update install_local_hadoop commit 509fa69f3428926482b2c95c334d4530d3ca3148 Author: mashengchen <mashengc...@gmail.com> Date: 2016-04-06T09:58:46Z Merge pull request #14 from apache/master update head --- 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-1917] Trafodion fails...
Github user mashengchen commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/415#discussion_r58515055 --- Diff: core/sqf/sql/scripts/install_local_hadoop --- @@ -614,7 +614,7 @@ HIVE_TAR=${HIVE_PREFIX}.tar.gz HBASE_MIRROR_URL=http://archive.cloudera.com/cdh5/cdh/5 HBASE_TAR=hbase-${HBASE_DEP_VER_CDH}.tar.gz if [[ "$HBASE_DISTRO" = "HDP" ]]; then -HBASE_TAR=hbase-${HBASE_DEP_VER_HDP}.tar.gz +HBASE_TAR=hbase-1.1.2.2.3.2.0-2950.tar.gz --- End diff -- agree with that, but how to decide the version for HBASE_DEP_VER_HDP, if it's a simplified version like 1.1.2, how would a new trafodion user know the actually version. or always use a fix version, like 1.1.2.2.3.2.0-2950? --- 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-1917] Trafodion fails...
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/415 [TRAFODION-1917] Trafodion fails to build HDP related jar files You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/415.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 #415 commit 0dbf1d1bc8718990850b4cff6e5f955a213209ae Author: mashengchen <mashengc...@gmail.com> Date: 2016-04-05T03:03:48Z Trafodion building troubles commit c0b3136c6e29afc97d299cfc3407df7093a20060 Author: mashengchen <mashengc...@gmail.com> Date: 2016-04-05T03:07:20Z Merge branch 'master' of github.com:mashengchen/incubator-trafodion commit 897ea4b4013e55c691ffabb84fef804c7a623bf8 Author: mashengchen <mashengc...@gmail.com> Date: 2016-04-05T03:09:01Z Merge pull request #13 from apache/master update head --- 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
Github user mashengchen commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/348#discussion_r54833403 --- Diff: core/sql/pom.xml.apache --- @@ -0,0 +1,131 @@ + --- End diff -- no idea to use one pom.xml, we have to make 3 version (vanilla,cdh,hdp), so have to use three pom.xml --- 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
Github user mashengchen commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/348#discussion_r54833294 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/regionserver/transactional/TrxTransactionState.java.tmpl --- @@ -77,96 +94,29 @@ * Holds the state of a transaction. This includes a buffer of all writes, a record of all reads / scans, and * information about which other transactions we need to check against. */ -public class TrxTransactionState extends TransactionState{ - -static boolean sb_sqm_98_1 = false; -static boolean sb_sqm_98_4 = false; -static boolean sb_sqm_98_9 = false; -static java.lang.reflect.Constructor c98_1 = null; -static java.lang.reflect.Constructor c98_4 = null; -static java.lang.reflect.Constructor c98_9 = null; +public class TrxTransactionState extends TransactionState { -static Class keepDeletedCellsClazz = null; -static Class scaninfoClazz = null; -static Constructor scaninfoConstructor = null; -static Object[] scaninfoArgs = null; +static java.lang.reflect.Constructor c1_0 = null; static { String version = VersionInfo.getVersion();// the hbase version string, eg. "0.6.3-dev" LOG.info("Got info of Class ScanQueryMatcher for HBase version :" + version); - --- End diff -- i don't think there will happen this rid.. cause we have a 1.3 branch to support backward version(hbase 0.98) --- 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
Github user mashengchen commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/348#discussion_r54819507 --- Diff: core/sqf/sql/scripts/install_local_hadoop --- @@ -579,17 +591,15 @@ fi # # See JIRA TRAFODION-1512 that will eliminate this check -if [ -d $MY_LOCAL_SW_DIST ]; then - HADOOP_MIRROR_URL=http://archive.cloudera.com/cdh5/cdh/5 - HADOOP_TAR=hadoop-2.5.0-cdh5.3.0.tar.gz -else - HADOOP_MIRROR_URL=http://download.nextag.com/apache/hadoop/common/hadoop-2.5.2 - HADOOP_TAR=hadoop-2.5.2.tar.gz -fi +HADOOP_MIRROR_URL=http://archive.cloudera.com/cdh5/cdh/5 +HADOOP_TAR=hadoop-2.6.0-cdh5.4.4.tar.gz -if [[ "$SQ_HBASE_DISTRO" = "HDP" ]]; then -HADOOP_TAR=hadoop-2.6.0.2.2.0.0-2041.tar.gz +if [[ "$HBASE_DISTRO" = "HDP" ]]; then --- End diff -- it's just for someone why want to install hbase version beside CDH. they can config it by do 'export HBASE_DISTRO=APACHE/HDP'. --- 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
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/348 TRAFODION-1706 support vanilla hbase1.0 & cdh 1.0.0-cdh5.4.4 You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/348.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 #348 commit 4ed6be8ceabeb10418ee7101081fe6fd1c02a12c Author: mashengchen <mashengc...@gmail.com> Date: 2016-02-25T03:33:39Z TRAFODION-1706 commit 451df06d92f46bdf1fda61c080df05c272a5b56d Author: mashengchen <mashengc...@gmail.com> Date: 2016-02-25T06:14:06Z Update Makefile commit d5a4882287a25e0f73674fe6029c503c601a6a58 Author: mashengchen <mashengc...@gmail.com> Date: 2016-02-25T06:32:16Z remove some echo commit f547a6095b9d8c157a0273a9e9605b386fa6f440 Author: mashengchen <mashengc...@gmail.com> Date: 2016-02-25T07:03:02Z update install_local_hadoop commit 2a842bcf3eea4984ed20cfef8f6d77c7a81d00ea Author: mashengchen <mashengc...@gmail.com> Date: 2016-02-29T03:04:16Z update hbase_utilities & add hdp support --- 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-1846
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/337 TRAFODION-1846 add traf_coprocessor.properties to all node . You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/337.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 #337 commit 9a573e9946d247a41ef15df9fd08d23fe9a80890 Author: mashengchen <mashengc...@gmail.com> Date: 2016-02-03T03:28:54Z Merge pull request #6 from apache/master update head commit 6cfb920dc9e69b51866547373354f6d9291757a4 Author: mashengchen <mashengc...@gmail.com> Date: 2016-02-14T07:03:25Z Merge pull request #7 from apache/master update head commit ca5dcc7858fbc93078a4ef4421aef972b156a42b Author: mashengchen <mashengc...@gmail.com> Date: 2016-02-23T08:26:55Z Merge pull request #8 from apache/master update head commit 5e8542561874de0a972e5b83bffdf9ffb3e84a2b Author: mashengchen <mashengc...@gmail.com> Date: 2016-02-23T08:37:20Z TRAFODION-1846 add traf_coprocessor.properties to add node. --- 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: support vanilla hbase 0.98.10
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/275 support vanilla hbase 0.98.10 change getFamilyMap() to getFamilyCellMap() add ScanQueryMatcher instantiation for hbase 0.98.9 and later version change ScanInfo instantiation for hbase 0.98.8 and later version You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion release1.3 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/275.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 #275 commit cd5fd033ee10694e3d76602099eef9a7581865a5 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-21T05:01:29Z support vanilla hbase 0.98.10 --- 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-1729
Github user mashengchen closed the pull request at: https://github.com/apache/incubator-trafodion/pull/258 --- 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-1729
Github user mashengchen closed the pull request at: https://github.com/apache/incubator-trafodion/pull/258 --- 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-1729
GitHub user mashengchen reopened a pull request: https://github.com/apache/incubator-trafodion/pull/258 TRAFODION-1729 change the coprocessor deployment method, to avoid intrusive into underlying HBase system. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/258.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 #258 commit 12d7342bb3d63991d31933139f5b3fee216ab8b7 Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-14T06:16:25Z Merge pull request #1 from apache/master update head commit 3450fb15d4ae86c20c0a31f7e0750709f433af59 Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-21T05:26:00Z Merge pull request #2 from apache/master update head commit c8fef619f858ff24b8076e4abc2f556d9fb42acb Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-23T13:09:41Z Merge pull request #3 from apache/master update commit 0042344cfb67900a1c815da932e5ea4c7ba93a19 Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-30T10:32:40Z Merge pull request #4 from apache/master update head commit 11f9686b5df87f70b79edac5621da0610c7e70b8 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T02:49:44Z jira1729 commit cac96bc7ea69a3f1293e9ddef5e752f8785fb886 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T03:19:05Z Merge pull request #5 from apache/master update head commit 62237083f0db8230e257b6a8a12b5ca21c3831d8 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T04:03:24Z Merge branch 'master' of github.com:apache/incubator-trafodion commit 5292fdc470db2317f4ab1133d298e98b97992e1e Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T04:19:31Z Merge branch 'master' of github.com:mashengchen/incubator-trafodion commit 0ea8552120bd12d8c5ed1757325276cca3eac172 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T05:39:43Z Update HBaseClient.java remove unnecessary log. commit 6e0193a1837148fe5fecd3653402ddaff61f54ce Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T05:41:00Z Update CoprocessorUtils.java change log level commit 0dbaf3b3164069587a854968869211d1b69c3434 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T05:52:45Z jira1729 commit d8b594f2b585de224a007e48ba735f0f5db860be Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-12T04:15:12Z add copyright commit e1af6b9b94b237ab98c8e40677ada757a9b76f18 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-12T05:38:26Z rename & update code commit 411b703f557bdcff63b971261715d9a2caecb989 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-12T05:50:13Z Merge branch 'master' of github.com:apache/incubator-trafodion commit c1b8448c90533a5a2b039cce116ee57563c77299 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-12T05:54:31Z remove trafcoprocess.properties commit b8e6e9c920039c2fd7c0a175214b93ed2335d3e9 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-14T07:54:08Z Update traf_coprocessor.properties use TrxRegionEndpoint as default commit b01d8cb00abcfcadf48125649bdb69aac8616ffd Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-15T04:16:02Z Update CoprocessorUtils.java commit a005ebf5ccf75a374f034f8b63891a396c651ae9 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-15T06:18:14Z Update HBaseClient.java change CoprocessorUtils.addCoprocessor, add isMVCC as param commit d11e9d4ecf4263b7999d4bf8e6460ba78da04150 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-15T06:22:24Z Update CoprocessorUtils.java change addCoprocessor add isMVCC as param. commit 5751399d9334b77bbaca629b9c93d55355b49f00 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-15T06:27:01Z Update traf_coprocessor.properties make mvcc & sscc as a separate ones commit 379f8aefaf12476b8bc0f7960636a33b1b341449 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-15T06:28:45Z Update CoprocessorUtils.java --- 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-1729
GitHub user mashengchen reopened a pull request: https://github.com/apache/incubator-trafodion/pull/258 TRAFODION-1729 change the coprocessor deployment method, to avoid intrusive into underlying HBase system. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/258.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 #258 commit 12d7342bb3d63991d31933139f5b3fee216ab8b7 Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-14T06:16:25Z Merge pull request #1 from apache/master update head commit 3450fb15d4ae86c20c0a31f7e0750709f433af59 Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-21T05:26:00Z Merge pull request #2 from apache/master update head commit c8fef619f858ff24b8076e4abc2f556d9fb42acb Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-23T13:09:41Z Merge pull request #3 from apache/master update commit 0042344cfb67900a1c815da932e5ea4c7ba93a19 Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-30T10:32:40Z Merge pull request #4 from apache/master update head commit 11f9686b5df87f70b79edac5621da0610c7e70b8 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T02:49:44Z jira1729 commit cac96bc7ea69a3f1293e9ddef5e752f8785fb886 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T03:19:05Z Merge pull request #5 from apache/master update head commit 62237083f0db8230e257b6a8a12b5ca21c3831d8 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T04:03:24Z Merge branch 'master' of github.com:apache/incubator-trafodion commit 5292fdc470db2317f4ab1133d298e98b97992e1e Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T04:19:31Z Merge branch 'master' of github.com:mashengchen/incubator-trafodion commit 0ea8552120bd12d8c5ed1757325276cca3eac172 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T05:39:43Z Update HBaseClient.java remove unnecessary log. commit 6e0193a1837148fe5fecd3653402ddaff61f54ce Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T05:41:00Z Update CoprocessorUtils.java change log level commit 0dbaf3b3164069587a854968869211d1b69c3434 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T05:52:45Z jira1729 --- 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-1729
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/258 TRAFODION-1729 change the coprocessor deployment method, to avoid intrusive into underlying HBase system. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/258.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 #258 commit 12d7342bb3d63991d31933139f5b3fee216ab8b7 Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-14T06:16:25Z Merge pull request #1 from apache/master update head commit 3450fb15d4ae86c20c0a31f7e0750709f433af59 Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-21T05:26:00Z Merge pull request #2 from apache/master update head commit c8fef619f858ff24b8076e4abc2f556d9fb42acb Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-23T13:09:41Z Merge pull request #3 from apache/master update commit 0042344cfb67900a1c815da932e5ea4c7ba93a19 Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-30T10:32:40Z Merge pull request #4 from apache/master update head commit 11f9686b5df87f70b79edac5621da0610c7e70b8 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T02:49:44Z jira1729 commit cac96bc7ea69a3f1293e9ddef5e752f8785fb886 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T03:19:05Z Merge pull request #5 from apache/master update head commit 62237083f0db8230e257b6a8a12b5ca21c3831d8 Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T04:03:24Z Merge branch 'master' of github.com:apache/incubator-trafodion commit 5292fdc470db2317f4ab1133d298e98b97992e1e Author: mashengchen <mashengc...@gmail.com> Date: 2016-01-11T04:19:31Z Merge branch 'master' of github.com:mashengchen/incubator-trafodion --- 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-1650
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/221 TRAFODION-1650 after hbase-0.98.9(include 0.98.9), the instantiation process of ScanQueryMatcher(class) is different from before, which add one long class as parameter. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/221.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 #221 commit 810003ec3729b60dc85d5d5b3303b200b7222b0c Author: mashengchen <mashengc...@gmail.com> Date: 2015-12-10T08:50:34Z TRAFODION-1650 --- 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-1643
Github user mashengchen closed the pull request at: https://github.com/apache/incubator-trafodion/pull/191 --- 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-1643
GitHub user mashengchen opened a pull request: https://github.com/apache/incubator-trafodion/pull/193 TRAFODION-1643 after hbase-0.98.8(include 0.98.8), the instantiation process of ScanInfo(class) is different from before, which need KeepDeletedCells(enum) as parameter instead of boolean. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mashengchen/incubator-trafodion master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-trafodion/pull/193.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 #193 commit 36a0be0a9a6eb5c23f007331ac61bbba217f612b Author: mashengchen <mashengc...@gmail.com> Date: 2015-11-13T08:09:10Z Merge pull request #1 from apache/master update head commit 4789f679c965b220e2f3e51e0ac7d6231cebfe46 Author: mashengchen <mashengc...@gmail.com> Date: 2015-11-26T07:27:37Z Merge pull request #2 from apache/master update head commit f918e29224c2d74ba6a1abb445d5ac27c17252cc Author: traf <traf@esgvm-2.novalocal> Date: 2015-11-26T07:46:12Z TRAFODION-1643 update ScanInfo instantiation to V0.98.8 commit 3b7b50650a5983586f15587bfb54df89102733b7 Author: mashengchen <mashengc...@gmail.com> Date: 2015-11-26T07:54:03Z Update TrxTransactionState.java commit 032f698720f953b467da8ddf219726b159051959 Author: mashengchen <mashengc...@gmail.com> Date: 2015-11-27T03:20:01Z Update Makefile commit 44400d4caef84838d38e346daae4a991a3e7a8e1 Author: mashengchen <mashengc...@gmail.com> Date: 2015-11-27T03:22:44Z Update sqenvcom.sh commit 24dab807fb549209c8afb874650099ca653cd811 Author: mashengchen <mashengc...@gmail.com> Date: 2015-11-27T03:23:24Z Update Makefile commit 684b2858301fe971232fd83c9fcb5dbbf17d99b9 Author: mashengchen <mashengc...@gmail.com> Date: 2015-11-27T03:24:29Z Update Makefile commit 0744e17ddee5a6bc0dee5fc33466f584947562f9 Author: mashengchen <mashengc...@gmail.com> Date: 2015-11-27T03:25:01Z Update Makefile --- 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-1530]deprecated put.g...
Github user mashengchen commented on a diff in the pull request: https://github.com/apache/incubator-trafodion/pull/166#discussion_r44618328 --- Diff: core/sqf/src/seatrans/hbase-trx/src/main/java/org/apache/hadoop/hbase/regionserver/transactional/SingleVersionDeleteNotSupported.java --- @@ -53,13 +54,15 @@ public SingleVersionDeleteNotSupported() { * mechansim will currently treat DeleteColumn the same as Delete which could cause confusion. */ public static void validateDelete(final Delete delete) throws SingleVersionDeleteNotSupported { -Collection<List> values = delete.getFamilyMap().values(); -for (List value : values) { -for (KeyValue kv : value) { -if (Type.Delete.getCode() == kv.getType()) { +Collection<List> values = delete.getFamilyCellMap().values(); +for (List value : values) { +for (Cell cell : value) { +KeyValue kv = new KeyValue(cell); --- End diff -- got it --- 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. ---