> On Feb. 2, 2017, 9:17 p.m., Ashutosh Chauhan wrote:
> > ql/src/test/queries/clientpositive/udf_in.q, line 8
> > <https://reviews.apache.org/r/56240/diff/1/?file=1622497#file1622497line8>
> >
> > Is this a requirement now, that expressions must be in paranthesis ?
Yes, it it illegal to do so in postgres and oracle see below
horton=# select 1=1 in (true, false);
ERROR: operator does not exist: integer = boolean
LINE 1: select 1=1 in (true, false);
^
HINT: No operator matches the given name and argument type(s). You might need
to add explicit type casts.
- pengcheng
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56240/#review164017
-----------------------------------------------------------
On Feb. 2, 2017, 8:56 p.m., pengcheng xiong wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56240/
> -----------------------------------------------------------
>
> (Updated Feb. 2, 2017, 8:56 p.m.)
>
>
> Review request for hive and Ashutosh Chauhan.
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> HIVE-15388
>
>
> Diffs
> -----
>
> ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java 0f05160
> ql/src/java/org/apache/hadoop/hive/ql/parse/IdentifiersParser.g 43fd69e
>
> ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFInternalInterval.java
> fcf291a
> ql/src/test/org/apache/hadoop/hive/ql/parse/TestParseDriverIntervals.java
> e5db412
>
> ql/src/test/org/apache/hadoop/hive/ql/udf/generic/TestGenericUDFInternalInterval.java
> 8166bc0
> ql/src/test/queries/clientpositive/interval_alt.q a4fbe9f
> ql/src/test/queries/clientpositive/parenthese.q PRE-CREATION
> ql/src/test/queries/clientpositive/udf_in.q a7ce3c6
> ql/src/test/queries/clientpositive/udf_notop.q dceab7e
> ql/src/test/results/clientnegative/char_pad_convert_fail2.q.out 2d08fb3
> ql/src/test/results/clientnegative/invalid_select_expression.q.out 3650ca7
> ql/src/test/results/clientnegative/ptf_negative_DistributeByOrderBy.q.out
> a0f3fe2
> ql/src/test/results/clientnegative/ptf_negative_PartitionBySortBy.q.out
> a0f3fe2
> ql/src/test/results/clientnegative/ptf_window_boundaries.q.out 3808f2c
> ql/src/test/results/clientnegative/ptf_window_boundaries2.q.out 3808f2c
> ql/src/test/results/clientnegative/subquery_missing_from.q.out eaf7735
> ql/src/test/results/clientnegative/subquery_subquery_chain.q.out e661e1b
> ql/src/test/results/clientnegative/subquery_with_or_cond.q.out 4a9cc45
> ql/src/test/results/clientnegative/udf_in.q.out 773230b
> ql/src/test/results/clientpositive/llap/vector_between_in.q.out 74384a4
> ql/src/test/results/clientpositive/llap/vector_interval_arithmetic.q.out
> 391c775
> ql/src/test/results/clientpositive/parenthese.q.out PRE-CREATION
> ql/src/test/results/clientpositive/perf/query14.q.out 21cabbc
> ql/src/test/results/clientpositive/show_functions.q.out 3c9bb4a
> ql/src/test/results/clientpositive/spark/vector_between_in.q.out 07af2a3
> ql/src/test/results/clientpositive/udf_between.q.out 5b01554
> ql/src/test/results/clientpositive/udf_in.q.out da07f61
> ql/src/test/results/clientpositive/udf_notop.q.out 25483c7
> ql/src/test/results/clientpositive/vector_interval_arithmetic.q.out dd4b7d1
>
> Diff: https://reviews.apache.org/r/56240/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> pengcheng xiong
>
>
