You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by mu...@apache.org on 2021/01/21 19:54:48 UTC

[hive] branch master updated: HIVE-24672: compute_stats_long.q fails for wrong reasons (Mustafa Iman, reviewed by Attila Magyar and Laszlo Bodor)

This is an automated email from the ASF dual-hosted git repository.

mustafaiman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new e68cfc8  HIVE-24672: compute_stats_long.q fails for wrong reasons (Mustafa Iman, reviewed by Attila Magyar and Laszlo Bodor)
e68cfc8 is described below

commit e68cfc8eb039d9bb762d4c751d997ef2395b5a5c
Author: Mustafa Iman <mu...@gmail.com>
AuthorDate: Wed Jan 20 15:12:47 2021 -0800

    HIVE-24672: compute_stats_long.q fails for wrong reasons (Mustafa Iman, reviewed by Attila Magyar and Laszlo Bodor)
    
    Change-Id: I61d941dcbf86fb2dd45772fc658b3dc887325bd0
---
 ql/src/test/queries/clientnegative/compute_bit_vector_fm_limit.q   | 7 +++++++
 ql/src/test/queries/clientnegative/compute_stats_long.q            | 4 ++--
 ...{compute_stats_long.q.out => compute_bit_vector_fm_limit.q.out} | 2 +-
 ql/src/test/results/clientnegative/compute_stats_long.q.out        | 2 +-
 4 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/ql/src/test/queries/clientnegative/compute_bit_vector_fm_limit.q b/ql/src/test/queries/clientnegative/compute_bit_vector_fm_limit.q
new file mode 100644
index 0000000..dd43f26
--- /dev/null
+++ b/ql/src/test/queries/clientnegative/compute_bit_vector_fm_limit.q
@@ -0,0 +1,7 @@
+create table tab_int(a int);
+
+-- insert some data
+LOAD DATA LOCAL INPATH "../../data/files/int.txt" INTO TABLE tab_int;
+
+-- this should raise an error since the number of bit vectors has a hard limit at 1024
+select compute_bit_vector_fm(a, 10000) from tab_int;
diff --git a/ql/src/test/queries/clientnegative/compute_stats_long.q b/ql/src/test/queries/clientnegative/compute_stats_long.q
index 5974811..bf87393 100644
--- a/ql/src/test/queries/clientnegative/compute_stats_long.q
+++ b/ql/src/test/queries/clientnegative/compute_stats_long.q
@@ -3,5 +3,5 @@ create table tab_int(a int);
 -- insert some data
 LOAD DATA LOCAL INPATH "../../data/files/int.txt" INTO TABLE tab_int;
 
--- compute stats should raise an error since the number of bit vectors > 1024
-select compute_stats(a, 10000) from tab_int;
+-- this should raise an error since the number of bit vectors has a hard limit at 1024
+select compute_stats(a, 'fm', 10000) from tab_int;
diff --git a/ql/src/test/results/clientnegative/compute_stats_long.q.out b/ql/src/test/results/clientnegative/compute_bit_vector_fm_limit.q.out
similarity index 92%
copy from ql/src/test/results/clientnegative/compute_stats_long.q.out
copy to ql/src/test/results/clientnegative/compute_bit_vector_fm_limit.q.out
index 79f2146..0341682 100644
--- a/ql/src/test/results/clientnegative/compute_stats_long.q.out
+++ b/ql/src/test/results/clientnegative/compute_bit_vector_fm_limit.q.out
@@ -14,7 +14,7 @@ POSTHOOK: query: LOAD DATA LOCAL INPATH "../../data/files/int.txt" INTO TABLE ta
 POSTHOOK: type: LOAD
 #### A masked pattern was here ####
 POSTHOOK: Output: default@tab_int
-PREHOOK: query: select compute_stats(a, 10000) from tab_int
+PREHOOK: query: select compute_bit_vector_fm(a, 10000) from tab_int
 PREHOOK: type: QUERY
 PREHOOK: Input: default@tab_int
 #### A masked pattern was here ####
diff --git a/ql/src/test/results/clientnegative/compute_stats_long.q.out b/ql/src/test/results/clientnegative/compute_stats_long.q.out
index 79f2146..e6164eb 100644
--- a/ql/src/test/results/clientnegative/compute_stats_long.q.out
+++ b/ql/src/test/results/clientnegative/compute_stats_long.q.out
@@ -14,7 +14,7 @@ POSTHOOK: query: LOAD DATA LOCAL INPATH "../../data/files/int.txt" INTO TABLE ta
 POSTHOOK: type: LOAD
 #### A masked pattern was here ####
 POSTHOOK: Output: default@tab_int
-PREHOOK: query: select compute_stats(a, 10000) from tab_int
+PREHOOK: query: select compute_stats(a, 'fm', 10000) from tab_int
 PREHOOK: type: QUERY
 PREHOOK: Input: default@tab_int
 #### A masked pattern was here ####