Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1384
Do you have --enable-cassert enabled? e.g.
./configure --enable-debug --enable-cassert
I got such kind of errors:
`make[3]: Entering directory
`/tmp/build/78017950
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1384
@oushu1wangziming1 What platform are you using? I got the issue on Linux.
Or if it requires any new dependencies? Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1384
First I see there are compile issue in dbg mode, and I see unittest
failures. Please help to verify.
And please add Jira number to to commit message. Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1387
The Jenkins test seems have git clone issue, travis test passed.
@edespino @jiny2 @huor @interma Please help to review, thanks.
---
GitHub user radarwave opened a pull request:
https://github.com/apache/incubator-hawq/pull/1387
HAWQ-1647. Update HAWQ version from 2.3.0.0 to 2.4.0.0
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/radarwave/incubator-hawq HAWQ
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq-site/pull/17
Add @dyozie
---
GitHub user radarwave opened a pull request:
https://github.com/apache/incubator-hawq-site/pull/17
HAWQ-1638. Add how to verify downloaded files section, removed md5 keys.
Added a section about how to verify downloaded files.
Removed MD5 references.
You can merge this pull
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq-site/pull/16
Download links fix will be update later.
@changleicn @edespino @jiny2
Would you help to review? Thanks.
---
GitHub user radarwave opened a pull request:
https://github.com/apache/incubator-hawq-site/pull/16
HAWQ-1638. Correct typo and use full name for ASF products
Correct typo for madlib and use full name for ASF products with the
first/main references.
You can merge this pull request
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1375
@pengbinbin1 Is this PR created by accident? If true, please help to close
it. Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1373
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1370
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1315
Seems alter schema is disabled with intentional.
Anyone knows the background why we disabled it?
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1359
LGTM.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1355
+1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1341
Thanks @weinan003 . Great to see the first PR for vectorized execution,
pease check below comments:
1. I guess GNC should be GUC?
2. Please add Jira number to each commits.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1338
Thanks @jiny2 Great to see HAWQ version upgraded to 2.3.0.0.
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1287
@outofmem0ry The PR is merged, please close this PR. Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1336
LGTM
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1336
Thanks @weinan003 help handle the openssl version issue. I think we should
modify configure.in then generate configure.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1334
Any Jira number for this PR and commit? Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1329
@deem0n Thanks for the contribution, please help to close this PR since
it's already been merged.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1287
There are already two approved reviews. I will merge this PR soon.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1329
Thanks @deem0n , please add 'HAWQ-1579' to the front of commit message
title as well.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1329
@deem0n We need a jira number for each commit, please create a jira or add
the existing jira information? Thanks.
See: https://issues.apache.org/jira/projects/HAWQ
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1328
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1325
Merged, please close this PR. Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1324
Merged, please close this PR. Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1325
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1323
+1
As Travis used thrift 0.10 in the past, let's fix CI first, we will check
thrift version later.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1318
@kroberts7 Thanks for your contribution.
Now the fix is merged to master, please close this PR. Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1313
Already merged, please close this PR. Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1312
Already merged, please close this PR. Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1292
Already merged, please close this PR. Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1317
LGTM
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1292
@kuien This PR passed all the tests in my environment.
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1292
Thanks @kuien .
I will test your PR in my environment since you are seeing some environment
issues in HAWQ-1558.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq-site/pull/14
LGTM +1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1313
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1312
LGTM.
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1292
@kuien Do you need any help to go through the test processes?
I think at least we need this PR pass the current feature tests, then you
can decide if a new test is needed
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1311
LGTM +1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1307
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1308
+1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1296
Thanks @edespino
So I'm good with it. +1
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1292
@kuien
Please refer to the README of feature test to add your test:
https://github.com/apache/incubator-hawq/tree/master/src/test/feature
And you can run all the feature
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1292
@kuien Have you ran and passed HAWQ feature tests for this fix? Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1296
Seems this is only for pxf main rpm, how about the other rpms related to
pxf?
pxf-hbase
pxf-hdfs
pxf-hive
pxf-jdbc
pxf-json
pxf-service
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1263
@outofmem0ry There is no jira number in your commit, I have add it and
merged this commit. Please close this PR. Thanks.
---
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1263
@outofmem0ry Please squash your commits and rewrite the commit messages as
now the answer file is not changed.
---
If your project is set up for it, you can reply to this email and have
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1267
@outofmem0ry Code merged, please close this PR. Thanks.
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1285
LGTM
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1284
+1
Thanks @outofmem0ry for your activated contribution.
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1267
LGTM
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1282
+1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1275
+1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1263
+1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1271
Travis have been broken for days, thanks @interma to bring it back.
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1270
LGTM +1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1268
Thanks @outofmem0ry
The commit is merged, please close this PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1268
Thanks @outofmem0ry to fix this.
LGTM +1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1227
When master is down, it's no matter to stop segment in 'smart' or 'fast'
mode. And I see this fix only affect 'hawq stop cluster', which means master is
already down before stoping
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1254
Thanks for @outofmem0ry 's contribution, I have squashed and merged this
commit. Please close this PR.
Welcome to do more contributions.
---
If your project is set up
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1254
LGTM +1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1264
+1
---
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
Github user radarwave commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1254#discussion_r125388848
--- Diff: src/test/feature/ManagementTool/test_hawq_reload.cpp ---
@@ -0,0 +1,52 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user radarwave commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1254#discussion_r125376981
--- Diff: tools/bin/hawq ---
@@ -120,6 +120,23 @@ def main():
if second_arg not in cluster_type_list:
print
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1254
@outofmem0ry
Please check the cases in below folder as example:
incubator-hawq/src/test/feature/ManagementTool
---
If your project is set up for it, you can reply to this email
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1258
LGTM. +1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1257
+1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1254
Thanks @outofmem0ry to contribute this, please check below comments.
1. I think we should keep the legacy command to do reload in case it's hard
coded in some user cases, print
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1252
LGTM. +1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1246
LGTM +1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1247
LGTM +1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1230
+1
---
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
Github user radarwave commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1211#discussion_r110341344
--- Diff: tools/bin/hawqpylib/hawqlib.py ---
@@ -203,6 +203,13 @@ def check_hostname_equal(remote_host, user = ""):
cmd =
Github user radarwave commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1211#discussion_r110324197
--- Diff: tools/bin/hawqpylib/hawqlib.py ---
@@ -203,6 +203,13 @@ def check_hostname_equal(remote_host, user = ""):
cmd =
Github user radarwave commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1211#discussion_r110322606
--- Diff: tools/bin/hawqpylib/hawqlib.py ---
@@ -203,6 +203,17 @@ def check_hostname_equal(remote_host, user = ""):
cmd =
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1203
LGTM. +1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1208
Thanks @shivzone .The virtual packages works to me.
+1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1208
sudo yum list installed | grep hadoop
hadoop-conf-pseudo.noarch2.7.3.2.5.0.0-1245.el6 @HDP-2.5.0.0
hadoop_2_5_0_0_1245.x86_64 2.7.3.2.5.0.0-1245.el6 @HDP
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1208
After set "export HD=hdp", the rpm package names changed back with the
version string "3_2_1_0". Maybe this is expected. But then I still seeing it
require
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1203
Now the rpm name is: hawq-ranger-plugin-2.2.0.0-1.el7.centos.noarch.rpm
How about we remove the string 'centos' since there is already a 'el7'
string there?
---
If your project
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1203
Now hawq ranger plugin rpm is in different directory, we should update the
tarball script 'contrib/hawq-package/make_rpm_tarball.sh' with the new copy
directory:
-cp
Github user radarwave commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1203#discussion_r108702120
--- Diff: ranger-plugin/pom.xml ---
@@ -38,6 +38,9 @@
1
9.1-901-1.jdbc4
UTF-8
Github user radarwave commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1207#discussion_r108593718
--- Diff: contrib/hawq-package/build_hawq_rpm.sh ---
@@ -71,8 +76,14 @@ rpmbuild --define "_topdir ${RPM_TOP_DIR}" \
GitHub user radarwave opened a pull request:
https://github.com/apache/incubator-hawq/pull/1207
HAWQ-326. Fix specified HAWQ_RELEASE_VERSION for rpm build.
This fix included:
1) Fixed HAWQ_RELEASE_VERSION for rpm tarball.
2) Make sure distclean delete all the rpm
Github user radarwave commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1203#discussion_r108583106
--- Diff: ranger-plugin/pom.xml ---
@@ -38,6 +38,9 @@
1
9.1-901-1.jdbc4
UTF-8
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1203
@ljainpivotalio So we need another jira to fix the dependence issue?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1203
Does this fixed below issue? Apache hawq rpm name should be 'apache-hawq',
not 'hawq_2_2_0_0'.
rpm -ivh hawq-ranger-plugin_2_2_0_0-2.2.0.0-1.el7.centos.noarch.rpm
error
Github user radarwave closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1195
---
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
Github user radarwave closed the pull request at:
https://github.com/apache/incubator-hawq/pull/1177
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1195
Made some changes base on @paul-guo- 's comments.
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1177
This PR is for build HAWQ rpm from binary tarball, since now we chose using
source tarball to build rpm packages, so open another PR for the review.
New PR link is:
https
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1195
@shivzone @denalex @huor @paul-guo- @edespino
Please help to review, including PXF/Ranger-plugin changes.
---
If your project is set up for it, you can reply to this email
GitHub user radarwave opened a pull request:
https://github.com/apache/incubator-hawq/pull/1195
HAWQ-326. Support RPM package for Apache HAWQ.
1. Add a RPM spec file to support building HAWQ rpm package from source
code tarball.
2. Add 'make rpm' command to build HAWQ/PXF
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1193
+1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1188
LGTM +1
---
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
Github user radarwave commented on the issue:
https://github.com/apache/incubator-hawq/pull/1188
May need to add one more file:
tools/bin/gppylib/data/2.2.json (can copy from
tools/bin/gppylib/data/2.1.json)
---
If your project is set up for it, you can reply to this email
1 - 100 of 188 matches
Mail list logo