Github user wengyanqing commented on the issue:
https://github.com/apache/incubator-hawq/pull/1347
LGTM
---
Github user wengyanqing commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1347#discussion_r175084597
--- Diff: contrib/vexecutor/tuplebatch.c ---
@@ -0,0 +1,188 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
Github user weinan003 commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1347#discussion_r175034657
--- Diff: contrib/vexecutor/tuplebatch.h ---
@@ -0,0 +1,52 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ *
Github user interma commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1347#discussion_r175022310
--- Diff: contrib/vexecutor/tuplebatch.h ---
@@ -0,0 +1,52 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or
Github user wengyanqing commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1347#discussion_r175018979
--- Diff: contrib/vexecutor/tuplebatch.c ---
@@ -0,0 +1,188 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
Github user wengyanqing commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1347#discussion_r175019315
--- Diff: contrib/vexecutor/tuplebatch.h ---
@@ -0,0 +1,52 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ *
Github user wengyanqing commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1347#discussion_r175015247
--- Diff: contrib/vexecutor/tuplebatch.c ---
@@ -0,0 +1,188 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+
Github user zhangjackey closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1345
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1344
@kapustor To get this PR merged, we need this PR have two '+1' from the
reviewer, and you need to create corresponding jira and include the jira number
in the title of the PR. You can
[
https://issues.apache.org/jira/browse/HAWQ-1598?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
WANG Weinan updated HAWQ-1598:
--
Fix Version/s: backlog
> Vectorized Scan Node Framework initialization
>
[
https://issues.apache.org/jira/browse/HAWQ-1598?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
WANG Weinan updated HAWQ-1598:
--
Component/s: Query Execution
> Vectorized Scan Node Framework initialization
>
[
https://issues.apache.org/jira/browse/HAWQ-1598?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
WANG Weinan reassigned HAWQ-1598:
-
Assignee: WANG Weinan (was: Radar Lei)
> Vectorized Scan Node Framework initialization
>
WANG Weinan created HAWQ-1598:
-
Summary: Vectorized Scan Node Framework initialization
Key: HAWQ-1598
URL: https://issues.apache.org/jira/browse/HAWQ-1598
Project: Apache HAWQ
Issue Type:
GitHub user weinan003 opened a pull request:
https://github.com/apache/incubator-hawq/pull/1347
HAWQ-1591 Common tuple batch structure for VecExe
Define *TupleBatchData* struct to hold a batch of tuples through vectorized
execution process.
You can merge this pull request
14 matches
Mail list logo
