[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Brock Noland (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274758#comment-14274758
 ] 

Brock Noland commented on HIVE-9257:


Thank you cleaning up my mess! :)

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274490#comment-14274490
 ] 

Szehon Ho commented on HIVE-9257:
-

[~gopalv] [~brocknoland] tests passed for HIVE-9340 and I submitted a new merge 
to trunk in HIVE-9352.  Will merge as soon as possible.  Sorry for 
inconvenience.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Brock Noland (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274399#comment-14274399
 ] 

Brock Noland commented on HIVE-9257:


Ahh yes, we discussed today that I forgot that in my patch. Nevermind me.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274390#comment-14274390
 ] 

Szehon Ho commented on HIVE-9257:
-

Yea I was going to reply earlier to Brock.  I submitted it for removal it now 
in HIVE-9340, it is one of the other cleanups, thats why I wanted to collect 
them and then merge.  I might be able to post a patch this evening.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Gopal V (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274386#comment-14274386
 ] 

Gopal V commented on HIVE-9257:
---

Thanks, [~brocknoland].

I traced the ec2 fetching to the root pom.xml, which has

{code}
 
   spark-snapshot
   
http://ec2-50-18-79-139.us-west-1.compute.amazonaws.com/data/spark_2.10-1.2-SNAPSHOT/
   
...
{code}

I don't see that removed with HIVE-7674 (or in the branch) - 
https://github.com/apache/hive/blob/spark/pom.xml#L217

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Brock Noland (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274314#comment-14274314
 ] 

Brock Noland commented on HIVE-9257:


[~szehon] - do you think we could merge just that one change?

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274284#comment-14274284
 ] 

Szehon Ho commented on HIVE-9257:
-

Yea that will be removed in next merge.  It's already checked in for HIVE-7674 
and will be done in HIVE-9352

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Gopal V (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274279#comment-14274279
 ] 

Gopal V commented on HIVE-9257:
---

Can you remove all the EC2 random references from the poms? My builds are 
trying to pull hive JARs from some random EC2 hosts.

{code}
Downloading: 
http://ec2-50-18-79-139.us-west-1.compute.amazonaws.com/data/spark_2.10-1.2-SNAPSHOT/org/apache/hive/hive-service/0.15.0-SNAPSHOT/maven-metadata.xml
{code}

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274185#comment-14274185
 ] 

Szehon Ho commented on HIVE-9257:
-

Thanks, +1 on moving the link to under AdminDocs.  

There might be some parts that should be in UserDocs, but at this point I am 
not sure which (if any) to put in there.  Maybe some of the recommended 
configuration, as other configurations are under "UserDocs".

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274161#comment-14274161
 ] 

Szehon Ho commented on HIVE-9257:
-

Yea sorry thats the way it went about the weekend.  We made the JIRA on Monday 
for people to know the intention.  The code cleanup took all week, longer than 
I anticipated due to state of spark-branch, and first patch was linked early on 
Friday during the workday.  And as Brock said, it is really hard to merge such 
a big branch during the week when trunk is in constant motion (tons of 
conflicts), making the weekend the only viable choice to do it.

Also its my fault about missing the snapshot as part of the patch, it is fixed 
already in HIVE 7674 last week and will be merged in next merge (I missed it 
not being in this merge).  I could fix it immediately too, but was going to 
collect some more fixes and then merge by tomorrow/Wed.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Brock Noland (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274100#comment-14274100
 ] 

Brock Noland commented on HIVE-9257:


Also, obviously, if there is any additional feedback on the work, we'd be happy 
discuss.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Brock Noland (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274088#comment-14274088
 ] 

Brock Noland commented on HIVE-9257:


Hi Sergey,

The change was created a many days ago. It's exceedingly hard to do a merge 
during the week due to commit volume. W.r.t Spark SNAPSHOT, we have depended on 
Calcite SNAPSHOT for a long time, even sadly in the 0.14.0 release. However, I 
agree a SNAPSHOT dep in trunk is not ideal and we actually have a change to 
resolve this committed to spark and we'll be merging that soon. I actually 
thought that was getting merged this time.

Brock

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-12 Thread Sergey Shelukhin (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14274083#comment-14274083
 ] 

Sergey Shelukhin commented on HIVE-9257:


It's not very nice to do such a large branch merge over the weekend with no 
advance warning.
This appears to depend on spark build snapshot; this is a bad practice in 
general, and I don't think it would be acceptable to have a Hive release with 
such dependency.
Is it possible to change to a released version of Spark?

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-11 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14273295#comment-14273295
 ] 

Szehon Ho commented on HIVE-9257:
-

Thanks Brock.  FYI, I am planning another merge in next few days to incorporate 
the review fixes, and also to get rid of the spark-snapshot dependency.  It 
will hopefully be a more managable patch that can uploaded this time and tested 
the normal way :)

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-11 Thread Brock Noland (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14273258#comment-14273258
 ] 

Brock Noland commented on HIVE-9257:


I also created HIVE-9344 to fix that optimize_nullscan tests that are so flaky.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-11 Thread Brock Noland (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14273256#comment-14273256
 ] 

Brock Noland commented on HIVE-9257:


The only test which failed and is not flaky is was the windowing test. I 
created HIVE-9343 to track the issue.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-11 Thread Brock Noland (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14273223#comment-14273223
 ] 

Brock Noland commented on HIVE-9257:


FYI - I am running a build here: 
http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/HIVE-TRUNK-HADOOP-2/1/
 to ensure all the tests are passing post merge. If there is any failing tests, 
due to the merge, we should address them ASAP tomorrow.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-11 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14273209#comment-14273209
 ] 

Szehon Ho commented on HIVE-9257:
-

Thanks I noticed, was collecting those as well as some other items I saw: 
HIVE-9340

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-11 Thread Xuefu Zhang (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14273177#comment-14273177
 ] 

Xuefu Zhang commented on HIVE-9257:
---

Actually my comments on RB were not covered by HIVE-9335, which already has +1 
pending. We may need a separate JIRA to cover them.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>  Labels: TODOC15
> Fix For: 0.15.0
>
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-10 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272823#comment-14272823
 ] 

Szehon Ho commented on HIVE-9257:
-

Latest test results looks much better:  
[http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build-Upload/6/|http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build-Upload/6/]

windowing.q has been failing for spark for some time, and is tracked to be 
fixed (HIVE-9104), should be soon.  The other test failure does not look 
related.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
> Attachments: trunk-mr2-spark-merge.properties
>
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-10 Thread Xuefu Zhang (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272801#comment-14272801
 ] 

Xuefu Zhang commented on HIVE-9257:
---

I reviewed most of patches in Spark branch during the months, and also produced 
some. I reviewed the mega patch here, and left a couple of comments on RB. 
However, these can be addressed as followups.

+1 pending on test.




> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-10 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272800#comment-14272800
 ] 

Szehon Ho commented on HIVE-9257:
-

For reference, URL of new patch.  Only difference is that the new spark.out 
files have updated file sizes: alter_merge_orc.q.out, 
alter_merge_stats_orc.q.out, and vectorized_ptf.q.out.

 
[http://ec2-50-18-79-139.us-west-1.compute.amazonaws.com/data/spark-to-trunk-merge/HIVE-9257.3.patch|http://ec2-50-18-79-139.us-west-1.compute.amazonaws.com/data/spark-to-trunk-merge/HIVE-9257.3.patch]

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-10 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272789#comment-14272789
 ] 

Szehon Ho commented on HIVE-9257:
-

Analyzing the test failures, the orc file sizes are reported different per OS, 
so regenerating some golden files on Ubuntu and trying again.  This should 
probably be looked at.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-10 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272776#comment-14272776
 ] 

Szehon Ho commented on HIVE-9257:
-

Yea to get 
[http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build-Upload/5/|http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build-Upload/5/]
 to run spark tests, I had to create a new build profile 
(trunk-mr2-spark-merge) with the spark test-properties.  Its a chicken-egg 
problem (see HIVE-7801), as I tried changing the main build properties 
(trunk-mr2) but that screwed up all the other precommit builds 

Anyway the last step post merge will be putting the spark properties in 
trunk-mr2.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-10 Thread Brock Noland (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272774#comment-14272774
 ] 

Brock Noland commented on HIVE-9257:


I am very familiar with this work. We have some follow-on jiras like HIVE-9335 
but that shouldn't hold the merge as we've done in the past.

+1 pending tests to merge this to trunk

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-10 Thread Brock Noland (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272754#comment-14272754
 ] 

Brock Noland commented on HIVE-9257:


I created HIVE-9335 and put a patch up for the review items I saw.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-10 Thread Brock Noland (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272751#comment-14272751
 ] 

Brock Noland commented on HIVE-9257:


Nice, only seven failed tests:

http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build-Upload/5/



> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-09 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272350#comment-14272350
 ] 

Szehon Ho commented on HIVE-9257:
-

For reference, patch is uploaded here:  
[http://ec2-50-18-79-139.us-west-1.compute.amazonaws.com/data/spark-to-trunk-merge/HIVE-9257.patch|http://ec2-50-18-79-139.us-west-1.compute.amazonaws.com/data/spark-to-trunk-merge/HIVE-9257.patch]
 to get around the JIRA 10MB restriction.

Ran a separate build that uses this patch, and the existing tests are passing:  
[http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build-Upload/lastCompletedBuild/testReport/|http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build-Upload/lastCompletedBuild/testReport/].
  

But spark tests are not running as ptest-master is missing spark test 
properties.  Going to reconfigure the build machine with sparkCliDriver 
properties and give it another spin.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-09 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272107#comment-14272107
 ] 

Szehon Ho commented on HIVE-9257:
-

Adding review request: 
[https://reviews.apache.org/r/29787/|https://reviews.apache.org/r/29787/].

As the entire patch is too big, this shows the modified files.  These have been 
cleanuped as part of HIVE-9319, HIVE-9306, HIVE-9305.

The new files can be found here:  
[http://svn.apache.org/repos/asf/hive/branches/spark/|http://svn.apache.org/repos/asf/hive/branches/spark/]
 or 
[https://github.com/apache/hive/tree/spark|https://github.com/apache/hive/tree/spark]
 under:

data/conf/spark/
itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithLocalClusterSpark.java
itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestMultiSessionsHS2WithLocalClusterSpark.java
itests/qtest-spark/
ql/src/java/org/apache/hadoop/hive/ql/exec/SparkHashTableSinkOperator.java
ql/src/java/org/apache/hadoop/hive/ql/exec/spark/
ql/src/java/org/apache/hadoop/hive/ql/lib/TypeRule.java
ql/src/java/org/apache/hadoop/hive/ql/optimizer/SparkMapJoinProcessor.java
ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/GenSparkSkewJoinProcessor.java
ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/SparkCrossProductCheck.java
ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/SparkMapJoinResolver.java
ql/src/java/org/apache/hadoop/hive/ql/optimizer/spark/
ql/src/java/org/apache/hadoop/hive/ql/parse/spark/
ql/src/java/org/apache/hadoop/hive/ql/plan/SparkBucketMapJoinContext.java
ql/src/java/org/apache/hadoop/hive/ql/plan/SparkEdgeProperty.java
ql/src/java/org/apache/hadoop/hive/ql/plan/SparkHashTableSinkDesc.java
ql/src/java/org/apache/hadoop/hive/ql/plan/SparkWork.java
ql/src/java/org/apache/hadoop/hive/ql/stats/CounterStatsAggregatorSpark.java
ql/src/test/org/apache/hadoop/hive/ql/exec/spark/
ql/src/test/queries/clientpositive/auto_join_stats.q
ql/src/test/queries/clientpositive/auto_join_stats2.q
ql/src/test/queries/clientpositive/bucket_map_join_spark1.q
ql/src/test/queries/clientpositive/bucket_map_join_spark2.q
ql/src/test/queries/clientpositive/bucket_map_join_spark3.q
ql/src/test/queries/clientpositive/bucket_map_join_spark4.q
ql/src/test/queries/clientpositive/multi_insert_mixed.q
ql/src/test/queries/clientpositive/multi_insert_union_src.q
ql/src/test/queries/clientpositive/parallel_join0.q
ql/src/test/queries/clientpositive/parallel_join1.q
ql/src/test/queries/clientpositive/spark_test.q
ql/src/test/queries/clientpositive/udf_example_add.q
ql/src/test/results/clientpositive/auto_join_stats.q.out
ql/src/test/results/clientpositive/auto_join_stats2.q.out
ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out
ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out
ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out
ql/src/test/results/clientpositive/bucket_map_join_spark4.q.out
ql/src/test/results/clientpositive/multi_insert_mixed.q.out
ql/src/test/results/clientpositive/multi_insert_union_src.q.out
ql/src/test/results/clientpositive/parallel_join0.q.out
ql/src/test/results/clientpositive/parallel_join1.q.out
ql/src/test/results/clientpositive/spark/
ql/src/test/results/clientpositive/spark_test.q.out
ql/src/test/results/clientpositive/udf_example_add.q.out
spark-client/

Cleanup and review of those have been done as part of HIVE-9281 and HIVE-9288.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HIVE-9257) Merge from spark to trunk January 2015

2015-01-09 Thread Szehon Ho (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-9257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14271918#comment-14271918
 ] 

Szehon Ho commented on HIVE-9257:
-

Actually this is not going to work, the other patch is still larger.  Need to 
think of another way.

> Merge from spark to trunk January 2015
> --
>
> Key: HIVE-9257
> URL: https://issues.apache.org/jira/browse/HIVE-9257
> Project: Hive
>  Issue Type: Sub-task
>  Components: Spark
>Affects Versions: 0.15.0
>Reporter: Szehon Ho
>Assignee: Szehon Ho
>
> The hive on spark work has reached a point where we can merge it into the 
> trunk branch.  Note that spark execution engine is optional and no current 
> users should be impacted.
> This JIRA will be used to track the merge.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)