Skip to content

Commit 9962dd2

Browse files
committed
between tests
1 parent af0b6f2 commit 9962dd2

File tree

3 files changed

+213
-197
lines changed

3 files changed

+213
-197
lines changed

src/to_substrait.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -561,9 +561,9 @@ void DuckDBToSubstrait::TransformExpr(Expression &dexpr, substrait::Expression &
561561
case ExpressionType::COMPARE_NOT_DISTINCT_FROM:
562562
TransformComparisonExpression(dexpr, sexpr);
563563
break;
564-
case ExpressionType::COMPARE_BETWEEN:
565-
TransformBetweenExpression(dexpr, sexpr);
566-
break;
564+
// case ExpressionType::COMPARE_BETWEEN:
565+
// TransformBetweenExpression(dexpr, sexpr);
566+
// break;
567567
case ExpressionType::CONJUNCTION_AND:
568568
case ExpressionType::CONJUNCTION_OR:
569569
TransformConjunctionExpression(dexpr, sexpr, col_offset);

test/sql/test_between.test

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,14 @@
44

55
require substrait
66

7+
require tpcds
8+
9+
statement ok
10+
PRAGMA enable_verification
11+
12+
statement ok
13+
CALL dsdgen(sf=0.01)
14+
715
statement ok
816
PRAGMA enable_verification
917

@@ -12,3 +20,11 @@ create table t as select * from range(100) as t(x)
1220

1321
statement ok
1422
CALL get_substrait('select * from t where x BETWEEN 4 AND 6');
23+
24+
#Q 5 WITHOUT BETWEEN
25+
statement ok
26+
CALL get_substrait('SELECT Avg(ss_quantity), Avg(ss_ext_sales_price), Avg(ss_ext_wholesale_cost), Sum(ss_ext_wholesale_cost) FROM store_sales, store, customer_demographics, household_demographics, customer_address, date_dim WHERE s_store_sk = ss_store_sk AND ss_sold_date_sk = d_date_sk AND d_year = 2001 AND ( ( ss_hdemo_sk = hd_demo_sk AND cd_demo_sk = ss_cdemo_sk AND cd_marital_status = ''U'' AND cd_education_status = ''Advanced Degree'' AND ss_sales_price AND hd_dep_count = 3 ) OR ( ss_hdemo_sk = hd_demo_sk AND cd_demo_sk = ss_cdemo_sk AND cd_marital_status = ''M'' AND cd_education_status = ''Primary'' AND ss_sales_price AND hd_dep_count = 1 ) OR ( ss_hdemo_sk = hd_demo_sk AND cd_demo_sk = ss_cdemo_sk AND cd_marital_status = ''D'' AND cd_education_status = ''Secondary'' AND ss_sales_price AND hd_dep_count = 1 ) ) AND ( ( ss_addr_sk = ca_address_sk AND ca_country = ''United States'' AND ca_state IN ( ''AZ'', ''NE'', ''IA'' ) AND ss_net_profit ) OR ( ss_addr_sk = ca_address_sk AND ca_country = ''United States'' AND ca_state IN ( ''MS'', ''CA'', ''NV'' ) AND ss_net_profit ) OR ( ss_addr_sk = ca_address_sk AND ca_country = ''United States'' AND ca_state IN ( ''GA'', ''TX'', ''NJ'' ) AND ss_net_profit) ); ')
27+
28+
#Q 5 WITH BETWEEN
29+
statement ok
30+
CALL get_substrait('SELECT Avg(ss_quantity), Avg(ss_ext_sales_price), Avg(ss_ext_wholesale_cost), Sum(ss_ext_wholesale_cost) FROM store_sales, store, customer_demographics, household_demographics, customer_address, date_dim WHERE s_store_sk = ss_store_sk AND ss_sold_date_sk = d_date_sk AND d_year = 2001 AND ( ( ss_hdemo_sk = hd_demo_sk AND cd_demo_sk = ss_cdemo_sk AND cd_marital_status = ''U'' AND cd_education_status = ''Advanced Degree'' AND ss_sales_price BETWEEN 100.00 AND 150.00 AND hd_dep_count = 3 ) OR ( ss_hdemo_sk = hd_demo_sk AND cd_demo_sk = ss_cdemo_sk AND cd_marital_status = ''M'' AND cd_education_status = ''Primary'' AND ss_sales_price BETWEEN 50.00 AND 100.00 AND hd_dep_count = 1 ) OR ( ss_hdemo_sk = hd_demo_sk AND cd_demo_sk = ss_cdemo_sk AND cd_marital_status = ''D'' AND cd_education_status = ''Secondary'' AND ss_sales_price BETWEEN 150.00 AND 200.00 AND hd_dep_count = 1 ) ) AND ( ( ss_addr_sk = ca_address_sk AND ca_country = ''United States'' AND ca_state IN ( ''AZ'', ''NE'', ''IA'' ) AND ss_net_profit BETWEEN 100 AND 200 ) OR ( ss_addr_sk = ca_address_sk AND ca_country = ''United States'' AND ca_state IN ( ''MS'', ''CA'', ''NV'' ) AND ss_net_profit BETWEEN 150 AND 300 ) OR ( ss_addr_sk = ca_address_sk AND ca_country = ''United States'' AND ca_state IN ( ''GA'', ''TX'', ''NJ'' ) AND ss_net_profit BETWEEN 50 AND 250 ) ); ')

0 commit comments

Comments
 (0)