You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by ta...@apache.org on 2019/05/06 22:41:12 UTC
[impala] branch master updated (25db9ea -> ab47981)
This is an automated email from the ASF dual-hosted git repository.
tarmstrong pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git.
from 25db9ea IMPALA-8496: Fix flakiness of test_data_cache.py
new 03ab3a5 IMPALA-8499: avoid datetime.total_seconds() in test_insert_events
new ab47981 IMPALA-8488: Fix hardcoded path in Ranger E2E test on S3
The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
tests/authorization/test_ranger.py | 3 ++-
tests/custom_cluster/test_event_processing.py | 5 ++---
2 files changed, 4 insertions(+), 4 deletions(-)
[impala] 01/02: IMPALA-8499: avoid datetime.total_seconds() in
test_insert_events
Posted by ta...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
tarmstrong pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git
commit 03ab3a58df0716b1259c1067175ee096c37460c0
Author: Tim Armstrong <ta...@cloudera.com>
AuthorDate: Mon May 6 08:59:46 2019 -0700
IMPALA-8499: avoid datetime.total_seconds() in test_insert_events
This function was only added in Python 2.7.
Change-Id: I8e6e556d99d07c1f559a2097fbd634bfc5eaaa52
Reviewed-on: http://gerrit.cloudera.org:8080/13245
Reviewed-by: Todd Lipcon <to...@apache.org>
Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
tests/custom_cluster/test_event_processing.py | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/tests/custom_cluster/test_event_processing.py b/tests/custom_cluster/test_event_processing.py
index c936807..7984664 100644
--- a/tests/custom_cluster/test_event_processing.py
+++ b/tests/custom_cluster/test_event_processing.py
@@ -18,7 +18,6 @@
import pytest
import json
import time
-from datetime import datetime
from tests.common.environ import build_flavor_timeout
import requests
from tests.common.skip import SkipIfS3, SkipIfABFS, SkipIfADLS, SkipIfIsilon, SkipIfLocal
@@ -110,11 +109,11 @@ class TestEventProcessing(CustomClusterTestSuite):
"""
new_event_id = self.get_last_synced_event_id()
success = True
- start_time = datetime.now()
+ start_time = time.time()
while new_event_id - previous_event_id < 2:
new_event_id = self.get_last_synced_event_id()
# Prevent infinite loop
- time_delta = (datetime.now() - start_time).total_seconds()
+ time_delta = time.time() - start_time
if time_delta > 10:
success = False
break
[impala] 02/02: IMPALA-8488: Fix hardcoded path in Ranger E2E test
on S3
Posted by ta...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
tarmstrong pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git
commit ab47981098a6492bdc5c36c90fd9f742100298b5
Author: Austin Nobis <an...@cloudera.com>
AuthorDate: Fri May 3 11:59:24 2019 -0500
IMPALA-8488: Fix hardcoded path in Ranger E2E test on S3
A hardcoded path in test_ranger.py for URI testing was updated to
support S3, local, and HDFS as opposed to just HDFS.
Testing:
- Ran authorization E2E tests
- Ran all FE tests
- Ran test_ranger.py with S3
Change-Id: Ie2c021ce212f483a644fdab4e77ab95031066b14
Reviewed-on: http://gerrit.cloudera.org:8080/13234
Reviewed-by: Impala Public Jenkins <im...@cloudera.com>
Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
tests/authorization/test_ranger.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/authorization/test_ranger.py b/tests/authorization/test_ranger.py
index b66cc58..cd9ff57 100644
--- a/tests/authorization/test_ranger.py
+++ b/tests/authorization/test_ranger.py
@@ -24,6 +24,7 @@ import requests
from getpass import getuser
from tests.common.custom_cluster_test_suite import CustomClusterTestSuite
+from tests.util.hdfs_util import NAMENODE
ADMIN = "admin"
RANGER_AUTH = ("admin", "admin")
@@ -261,7 +262,7 @@ class TestRanger(CustomClusterTestSuite):
result = self.client.execute("show grant {0} {1} on uri '{2}'"
.format(kw, id, uri))
TestRanger._check_privileges(result, [
- [kw, id, "", "", "", "hdfs://localhost:20500" + uri, "", "all", "false"]])
+ [kw, id, "", "", "", "{0}{1}".format(NAMENODE, uri), "", "all", "false"]])
# Revoke uri privileges and verify
admin_client.execute("revoke all on uri '{0}' from {1} {2}"