[GitHub] incubator-trafodion pull request #1343: [TRAFODION-2805] Add "or edition" to...

[sumory]

GitHub user sumory opened a pull request:

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

[TRAFODION-2805] Add "or edition" to error info 4222



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

$ git pull https://github.com/sumory/incubator-trafodion fix-TRAFODION-2805

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

https://github.com/apache/incubator-trafodion/pull/1343.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 #1343


commit 6e4437ab82abbf60670d26d00d39fc96d9456a35
Author: sumory.wu 
Date:   2017-12-14T03:43:33Z

[TRAFODION-2805] Add "or edition" to error info 4222




---

[GitHub] incubator-trafodion pull request #1342: [TRAFODION-2802] Prepare the build e...

[sumory]

GitHub user sumory opened a pull request:

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

[TRAFODION-2802] Prepare the build environment with one command



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

$ git pull https://github.com/sumory/incubator-trafodion fix-TRAFODION-2802

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

https://github.com/apache/incubator-trafodion/pull/1342.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 #1342


commit 4b632272419a862c21012619155e3645cb99
Author: sumory.wu 
Date:   2017-12-14T03:32:38Z

[TRAFODION-2802] Prepare the build environment with one command




---

[GitHub] incubator-trafodion pull request #1341: [TRAFODION-2733] BMO quota changes

[selvaganesang]

GitHub user selvaganesang opened a pull request:

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

[TRAFODION-2733] BMO quota changes

To ensure BMO quota assignment works even when the estimates are way off.

Introduced a CQD BMO_MEMORY_ESTIMATE_OUTLIER_FACTOR. When a BMO memory
estimate exceeds BMO_MEMORY_LIMIT_PER_NODE_IN_MB by this factor,
the BMO memory estimate for the operator is reset to
BMO_MEMORY_LIMIT_PER_NODE_IN_MB * BMO_MEMORY_ESTIMATE_OUTLIER_FACTOR.
Then BMO memory estimate and the quota is assigned based on the
revised estimate.

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

$ git pull https://github.com/selvaganesang/incubator-trafodion 
trafodion-2733

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

https://github.com/apache/incubator-trafodion/pull/1341.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 #1341


commit 375eeeab7a178348c5d3ac6e89e40ee8a06cb0d2
Author: selvaganesang 
Date:   2017-12-06T19:50:04Z

[TRAFODION-2733] BMO quota changes

To ensure BMO quota assignment works even when the estimates are way off.

Introduced a CQD BMO_MEMORY_ESTIMATE_OUTLIER_FACTOR. When a BMO memory
estimate exceeds BMO_MEMORY_LIMIT_PER_NODE_IN_MB by this factor,
the BMO memory estimate for the operator is reset to
BMO_MEMORY_LIMIT_PER_NODE_IN_MB * BMO_MEMORY_ESTIMATE_OUTLIER_FACTOR.
Then BMO memory estimate and the quota is assigned based on the
revised estimate.




---

[GitHub] incubator-trafodion pull request #1327: [TRAFODION-2803] add regression test...

[asfgit]

Github user asfgit closed the pull request at:

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


---

[GitHub] incubator-trafodion pull request #1330: [TRAFODION-2832] Fix for hang when p...

[asfgit]

Github user asfgit closed the pull request at:

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


---

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

[asfgit]

Github user asfgit closed the pull request at:

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


---

[GitHub] incubator-trafodion pull request #1311: [TRAFODION-2818] For Server 2008, fu...

[DaveBirdsall]

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


https://github.com/apache/incubator-trafodion/pull/1311#discussion_r153246992
  
--- Diff: win-odbc64/odbcclient/drvr35/sqltocconv.cpp ---
@@ -36,6 +36,34 @@
 
 #define MAXCHARLEN 32768 //32K
 
+// for server2008 when using function pow() then throws 
STATUS_ILLEGAL_INSTRUCTION
+__int64 pow(int base, short scale)
+{
+   DWORD dwVersion = 0;
+   DWORD dwBuild = 0;
+
+   dwVersion = GetVersion();
+
+   // Get the build number.
+
+   if (dwVersion < 0x8000)
+   dwBuild = (DWORD)(HIWORD(dwVersion));
+
+   __int64 retValue = 1;
+   if (dwBuild == 7600)
+   {
+   scale = scale > 18 ? 18 : scale;
+   for (int i = 0; i < scale; i++)
+   retValue = retValue * 10;
--- End diff --

So, if the scale was 19, we would return 10^18? Is this correct? Shouldn't 
there be some sort of overflow error instead?


---

[GitHub] incubator-trafodion pull request #1320: TRAFODION-2823 some test case donot ...

[DaveBirdsall]

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


https://github.com/apache/incubator-trafodion/pull/1320#discussion_r156799988
  
--- Diff: 
dcs/src/test/jdbc_test/src/test/java/org/trafodion/jdbc_test/TestNetworkTimeout.java
 ---
@@ -64,10 +72,19 @@ public void testGetNetworkTimeout() {
Connection conn = null;
try {
System.out.println("Connecting to database...");
-   conn = Utils.getUserConnection();
+   conn = Utils.getUserConnection();S
--- End diff --

The extra "S" at the end of the line appears to be causing the JDBC build 
failure


---

[GitHub] incubator-trafodion pull request #1323: [TRAFODIION-2828] Multi-threaded bat...

[asfgit]

Github user asfgit closed the pull request at:

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


---

[GitHub] incubator-trafodion pull request #1340: [TRAFODION-2843] Fix outer join on a...

[DaveBirdsall]

GitHub user DaveBirdsall opened a pull request:

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

[TRAFODION-2843] Fix outer join on aggregate bug

Formerly, the code was allowing an uncovered aggregate expression to be 
generated as a before join predicate. This lead to a Generator assert. The test 
for uncovered expressions in Join::preCodeGen has been tightened up so this 
does not happen.

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

$ git pull https://github.com/DaveBirdsall/incubator-trafodion Trafodion2843

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

https://github.com/apache/incubator-trafodion/pull/1340.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 #1340


commit ccb17c28e7fa74b5c58dbf37a02239316184bdd5
Author: Dave Birdsall 
Date:   2017-12-13T21:04:44Z

[TRAFODION-2843] Fix outer join on aggregate bug




---

[GitHub] incubator-trafodion pull request #1318: TRAFODION-2825 add jdbc user case

[asfgit]

Github user asfgit closed the pull request at:

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


---

[GitHub] incubator-trafodion pull request #1337: [TRAFODION-2838] Add error check to ...

[asfgit]

Github user asfgit closed the pull request at:

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


---