hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ashutosh Chauhan <hashut...@apache.org>
Subject Re: Review Request 56240: HiveParser spends lots of time in parsing queries with lots of "("
Date Thu, 02 Feb 2017 21:17:04 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56240/#review164017
-----------------------------------------------------------



Any reason for deleting interval_alt.q ?


ql/src/test/queries/clientpositive/udf_in.q (line 8)
<https://reviews.apache.org/r/56240/#comment235530>

    Is this a requirement now, that expressions must be in paranthesis ?


- Ashutosh Chauhan


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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message