[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-28 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. IMPALA-4847: Simplify HdfsTable block metadata loading code This commit is a part

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-28 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7652 To unsubsc

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-28 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1158/ --

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-28 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. Patch Set 4: Code-Review+2 Here is the perf analysis of this jira [1]. This regresses the S

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-28 Thread Bharath Vissapragada (Code Review)
Hello Dimitris Tsirogiannis, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7652 to look at the new patch set (#4). Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code

Re: [Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-11 Thread Dimitris Tsirogiannis
Sounds good to me. Dimitris On Fri, Aug 11, 2017 at 2:21 PM, Bharath Vissapragada (Code Review) < ger...@cloudera.org> wrote: > Bharath Vissapragada has posted comments on this change. > > Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code > .

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-11 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. Patch Set 3: Thanks Dimitris for the quick reviews. I'll run the metadata benchmark again

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-11 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7652 To unsu

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-11 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#3). Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. IMPALA-4847: Simplify HdfsTable block metadata loading code This commit is a part of ground w

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-11 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/7652/2/fe/src/main/java/org/

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-11 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. Patch Set 2: (3 comments) Yay, more lined deleted :) http://gerrit.cloudera.org:8080/#/c

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-11 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/7652/1/fe/src/main/java/org/

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-11 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#2). Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. IMPALA-4847: Simplify HdfsTable block metadata loading code This commit is a part of ground w

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-10 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/7652/1/fe/src/main/java/org

[Impala-ASF-CR] IMPALA-4847: Simplify HdfsTable block metadata loading code

2017-08-10 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new change for review. http://gerrit.cloudera.org:8080/7652 Change subject: IMPALA-4847: Simplify HdfsTable block metadata loading code .. IMPALA-4847: Simplify HdfsTable block metadata load