[jira] [Commented] (HIVE-10643) Refactoring Windowing for sum() to pass WindowFrameDef instead of two numbers (1 for number of preceding and 1 for number of following)

2015-05-12 Thread Aihua Xu (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-10643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14540265#comment-14540265 ] Aihua Xu commented on HIVE-10643: - Thanks. > Refactoring Windowing for sum() to pass Wind

[jira] [Commented] (HIVE-10643) Refactoring Windowing for sum() to pass WindowFrameDef instead of two numbers (1 for number of preceding and 1 for number of following)

2015-05-12 Thread Aihua Xu (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-10643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14539844#comment-14539844 ] Aihua Xu commented on HIVE-10643: - Unrelated failures. > Refactoring Windowing for sum()

[jira] [Commented] (HIVE-10643) Refactoring Windowing for sum() to pass WindowFrameDef instead of two numbers (1 for number of preceding and 1 for number of following)

2015-05-11 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-10643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14538817#comment-14538817 ] Hive QA commented on HIVE-10643: {color:red}Overall{color}: -1 at least one tests failed

[jira] [Commented] (HIVE-10643) Refactoring Windowing for sum() to pass WindowFrameDef instead of two numbers (1 for number of preceding and 1 for number of following)

2015-05-11 Thread Ashutosh Chauhan (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-10643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14538588#comment-14538588 ] Ashutosh Chauhan commented on HIVE-10643: - +1 > Refactoring Windowing for sum() t

[jira] [Commented] (HIVE-10643) Refactoring Windowing for sum() to pass WindowFrameDef instead of two numbers (1 for number of preceding and 1 for number of following)

2015-05-11 Thread Aihua Xu (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-10643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14538570#comment-14538570 ] Aihua Xu commented on HIVE-10643: - Thanks [~ashutoshc] for reviewing. Uploaded a new patch

[jira] [Commented] (HIVE-10643) Refactoring Windowing for sum() to pass WindowFrameDef instead of two numbers (1 for number of preceding and 1 for number of following)

2015-05-11 Thread Ashutosh Chauhan (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-10643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14538397#comment-14538397 ] Ashutosh Chauhan commented on HIVE-10643: - Mostly looks good. Left few comments on

[jira] [Commented] (HIVE-10643) Refactoring Windowing for sum() to pass WindowFrameDef instead of two numbers (1 for number of preceding and 1 for number of following)

2015-05-11 Thread Aihua Xu (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-10643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14537973#comment-14537973 ] Aihua Xu commented on HIVE-10643: - [~ashutoshc], can you help review the code? This is the

[jira] [Commented] (HIVE-10643) Refactoring Windowing for sum() to pass WindowFrameDef instead of two numbers (1 for number of preceding and 1 for number of following)

2015-05-11 Thread Aihua Xu (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-10643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14537955#comment-14537955 ] Aihua Xu commented on HIVE-10643: - All of the failures seem to be unrelated. > Refactori

[jira] [Commented] (HIVE-10643) Refactoring Windowing for sum() to pass WindowFrameDef instead of two numbers (1 for number of preceding and 1 for number of following)

2015-05-09 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-10643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14536731#comment-14536731 ] Hive QA commented on HIVE-10643: {color:red}Overall{color}: -1 at least one tests failed