Re: [VOTE] Graduate to a TLP

2017-10-17 Thread Matthew Jacobs
+1 On Tue, Oct 17, 2017 at 9:46 PM Sailesh Mukil wrote: > +1 > > On Tue, Oct 17, 2017 at 9:41 PM, Marcel Kornacker > wrote: > > > +1 > > > > On Tue, Oct 17, 2017 at 9:30 PM, Jeszy wrote: > > > +1 > > > > > > On 18 October 2017 at 03:40, Tim Armstrong > > wrote: > > >> +1 > > >> > > >> On 17 O

Re: Time for graduation?

2017-10-16 Thread Matthew Jacobs
+1 On Tue, Oct 17, 2017 at 4:59 AM Taras Bobrovytsky wrote: > +1 for graduation. I think Impala already lives by the ASF standards for > top level projects and is ready to graduate. > > On Mon, Oct 16, 2017 at 11:55 AM, Carl Steinbach wrote: > > > +1 for graduation > > > > On Mon, Oct 16, 2017

Re: New Impala committer: Joe McDonnell

2017-09-20 Thread Matthew Jacobs
Congrats, Joe! Well deserved. On Wed, Sep 20, 2017 at 2:25 PM, Alexander Behm wrote: > The Podling Project Management Committee (PPMC) for Apache Impala > (incubating) has invited Joe McDonnell to become a committer and we are > pleased to announce that he has accepted. > > Congratulations and we

Re: jenkins.impala.io pre-existing workspace

2017-09-19 Thread Matthew Jacobs
https://issues.apache.org/jira/browse/IMPALA-5959 On Tue, Sep 19, 2017 at 2:54 PM, Jim Apple wrote: > Nobody did. > > On Tue, Sep 19, 2017 at 2:29 PM, Matthew Jacobs wrote: > >> Did anyone file a JIRA for this? I saw this again. >> >> On Thu, Aug 31, 2017 at 1:36 P

Re: jenkins.impala.io pre-existing workspace

2017-09-19 Thread Matthew Jacobs
Did anyone file a JIRA for this? I saw this again. On Thu, Aug 31, 2017 at 1:36 PM, Jim Apple wrote: > Also, to be clear, I don't have the cycles to lead the fix-the-cleanup > task at the moment. > > On Wed, Aug 30, 2017 at 4:45 PM, Jim Apple wrote: >> The workspace cleanup isn't working - see t

Re: FYI gerrit outage next wednesday

2017-09-14 Thread Matthew Jacobs
+dev@impala On Thu, Sep 14, 2017 at 10:44 AM, Todd Lipcon wrote: > Gerrit.cloudera.org will be unavailable > on Wednesday > September 20th from 7:00-9:00PM PT. > > We will be upgrading from version 2.12.7 to 2.14.2. Patch notes >

Re: [VOTE] 2.10.0 release candidate 1 (RC1)

2017-08-28 Thread Matthew Jacobs
Bharath, is your key set up correctly? Running the script on https://cwiki.apache.org/confluence/display/IMPALA/How+to+Release#HowtoRelease-HowtoVoteonaReleaseCandidate resulted in this warning indicating that your signature is not trusted: gpg: WARNING: This key is not certified with a trusted s

Re: jenkins.impala.io outage tomorrow for instance size upgrade

2017-08-09 Thread Matthew Jacobs
Thanks, Michael! On Wed, Aug 9, 2017 at 9:21 AM, Michael Brown wrote: > jenkins.impala.io is back up and a test GVD has gotten far enough along > that everything seems to be in order.. Please let us know if you have any > problems. > > We now have more free memory than in use, which was absolutel

Re: Loading tpc-ds

2017-08-04 Thread Matthew Jacobs
sues.apache.org/jira/browse/IMPALA-5663 > > https://jenkins.impala.io/job/ubuntu-14.04-from-scratch/1827/artifact/Impala/logs_static/logs/cluster/hive/hive.log/*view*/ > > On Thu, Aug 3, 2017 at 9:09 PM, Matthew Jacobs wrote: > >> Just saw this error again. I filed IMPALA-5765. >

Re: Loading tpc-ds

2017-08-03 Thread Matthew Jacobs
Just saw this error again. I filed IMPALA-5765. On Mon, Jul 31, 2017 at 8:05 PM, Tim Armstrong wrote: > It looks like the same error: > > Caused by: org.apache.hadoop.hive.ql.metadata.HiveException: > org.apache.hadoop.ipc.RemoteException(java.io.IOException): File > /test-warehouse/tpcds.store_s

Re: Impala Insert Data Timed Out

2017-07-25 Thread Matthew Jacobs
Please send the profile. On Tue, Jul 25, 2017 at 7:07 PM sky wrote: > Hi all, > t1 tableļ¼š3000 data > t2 table: 0 data, the table structure is the same as t1 > using "insert into table t2 select * from t2;" will cause a timeout > error. Why ? -- Sent from my iPhone

Re: Slow/unusable apache JIRA?

2017-07-24 Thread Matthew Jacobs
Great, thanks On Mon, Jul 24, 2017 at 9:25 AM, Henry Robinson wrote: > There's a link here: https://www.apache.org/dev/infra-contact > > On 24 July 2017 at 09:24, Matthew Jacobs wrote: > > > Thanks. > > @Henry, where is the infra hipchat channel? > > > &g

Re: Slow/unusable apache JIRA?

2017-07-24 Thread Matthew Jacobs
Thanks. @Henry, where is the infra hipchat channel? On Mon, Jul 24, 2017 at 9:22 AM, Jeszy wrote: > Hey, > > No problems usually, but now It's down for me as well. According to > status.apache.org, the service seems to be struggling. > > On 24 July 2017 at 18:15, Matthew

Slow/unusable apache JIRA?

2017-07-24 Thread Matthew Jacobs
Hey, I've been noticing a lot of slowness/timeouts on the Apache JIRA. Has anyone else noticed this? Sometimes it's just annoying, but today I've found a lot of pages are just timing out. Just got this error when attempting to load https://issues.apache.org/jira/browse/IMPALA-5275 Communication

Re: Problem running Impala built with dynamic linking

2017-07-21 Thread Matthew Jacobs
Did you ever come to a conclusion on this? On Wed, Jul 19, 2017 at 5:14 PM, Bikramjeet Vig wrote: > Didn't work, heres the output of rm CMakeCache.txt && cmake . > > > -- Setup toolchain link flags > -Wl,-rpath,/home/bikram/dev/Impala/toolchain/gcc-4.9.2/lib64 > -L/home/bikram/dev/Impala/toolchai

Re: Is anyone else hitting a problem with Sentry when starting the Impala mini cluster?

2017-07-19 Thread Matthew Jacobs
+Bikram, who just got this in a gerrit-verify-dryrun job On Wed, Jul 19, 2017 at 4:32 PM, Taras Bobrovytsky wrote: > When I run ./testdata/bin/run-all.sh I get the following: > Error in Impala/testdata/bin/run-all.sh at line 58: > $IMPALA_HOME/testdata/bin/run-sentry-service.sh > > run-sentry-ser

Re: Rollback Mechanism

2017-07-13 Thread Matthew Jacobs
No, Kudu does not support multi-row transactions yet which will be needed to support "rolling back" failed data loads. Best, Matt On Wed, Jul 12, 2017 at 11:52 PM, sky wrote: > Hi all, > Does impala and kudu have rollback mechanism when load data ? > Thanks.

[hs2client-CR] Remove logging.h from client headers

2017-06-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: Remove logging.h from client headers .. Remove logging.h from client headers Remove logging.h from public headers to enable building in projects that

[hs2client-CR] Remove logging.h from client headers

2017-06-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Remove logging.h from client headers .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7199 To unsubscribe, visit http

[hs2client-CR] Remove logging.h from client headers

2017-06-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/7199 Change subject: Remove logging.h from client headers .. Remove logging.h from client headers Remove logging.h from public headers to

Re: [VOTE] 2.9.0 release candidate 1

2017-06-05 Thread Matthew Jacobs
+1 binding Downloaded the RC tar ball, verified checksums and signature. Ran the RAT script. Compiled the source tar locally, started the mini cluster, loaded a recent data snapshot, ran all tests. All green. On Sun, Jun 4, 2017 at 10:16 AM, Bharath Vissapragada wrote: > +1 (non-binding) > > Dow

Re: last_day() function

2017-05-23 Thread Matthew Jacobs
Yes, it'd be added to timestamp-functions.h in the BE and the symbol exposed in impala_functions.py. It looks like boost has a function called "end_of_month()": http://www.boost.org/doc/libs/1_64_0/doc/html/date_time/gregorian.html though I'm not sure if there are corner cases to consider. Best,

Re: Kerberos and RM

2017-05-15 Thread Matthew Jacobs
I'm not sure what that sentence is talking about either, but I know it doesn't apply to admission control (current supported RM feature). Seems like a good idea to remove it. On Mon, May 15, 2017 at 10:51 AM, Alexander Behm wrote: > Is that sentence even relevant? My guess is that "resource manag

Re: impala and yarn

2017-05-09 Thread Matthew Jacobs
r 4 impalad > as coordinator(in about 30 impalad cluster), I do not find any isolation about > CPU. does Impala community have some plans about this ? > > 2017-05-09 12:33 GMT+08:00 Matthew Jacobs : > >> Hi Yu, >> >> Yes, you can use the fair-scheduler.xml and llam

Re: impala and yarn

2017-05-08 Thread Matthew Jacobs
Hi Yu, Yes, you can use the fair-scheduler.xml and llama-site.xml files as you described. Admission control is the best mechanism that Impala has right now, unfortunately there isn't a way to get hard resource isolation between queries. I don't understand your problem though, are you saying that

Re: Need reviewer for distcc patches

2017-05-08 Thread Matthew Jacobs
I'll review it now. On Mon, May 8, 2017 at 8:51 AM, Tim Armstrong wrote: > This will unblock distcc for a decent number of people: > https://gerrit.cloudera.org/#/c/6655/ . It would be good to get it in > sooner rather than later. > > Cheers, > Tim

Re: free-pool-test BE failure in gerrit-verify-dryrun?

2017-05-05 Thread Matthew Jacobs
Does anyone know if there's a way to get core files from the gerrit-verify jobs? I can't reproduce this locally. On Fri, May 5, 2017 at 10:23 AM, Matthew Jacobs wrote: > I don't think the gerrit-verify-dryrun job saves them. > > On Fri, May 5, 2017 at 10:20 AM, Daniel H

Re: free-pool-test BE failure in gerrit-verify-dryrun?

2017-05-05 Thread Matthew Jacobs
I don't think the gerrit-verify-dryrun job saves them. On Fri, May 5, 2017 at 10:20 AM, Daniel Hecht wrote: > The core file doesn't get preserved somewhere? > > On Fri, May 5, 2017 at 10:10 AM, Matthew Jacobs wrote: > >> Has anyone seen this failure before? >>

free-pool-test BE failure in gerrit-verify-dryrun?

2017-05-05 Thread Matthew Jacobs
Has anyone seen this failure before? Start 18: free-pool-test 18/89 Test #18: free-pool-test ...***Exception: Other 11.05 sec Turning perftools heap leak checking off [==] Running 3 tests from 1 test case. [--] Global test environment set-up. [--] 3 te

Re: impala and yarn

2017-05-03 Thread Matthew Jacobs
Hi Yu, Llama was unsupported because it didn't work properly and we believe that the design was flawed. Instead, we recommend using a combination of "CM Static Service Pools" and Impala's admission control (as you mentioned). The best reference explaining this is: https://www.cloudera.com/documen

Re: jenkins will be restarted for upgrades

2017-04-10 Thread Matthew Jacobs
... and jenkins.impala.io is back On Mon, Apr 10, 2017 at 5:06 PM, Matthew Jacobs wrote: > I'll send an e-mail when done.

jenkins will be restarted for upgrades

2017-04-10 Thread Matthew Jacobs
I'll send an e-mail when done.

Re: Verification run for 6389

2017-04-07 Thread Matthew Jacobs
build is closed for anything except build blockers On Fri, Apr 7, 2017 at 1:30 PM, Zachary Amsden wrote: > Assuming things are good with the build, could someone kick off a > verification run for https://gerrit.cloudera.org/#/c/6389/ > > Thanks, > > - Zach >

Re: Kudu version mismatch between impala-config.sh and toolchain

2017-03-29 Thread Matthew Jacobs
https://gerrit.cloudera.org/#/c/6513/ On Wed, Mar 29, 2017 at 11:22 AM, Matthew Jacobs wrote: > Will do. > > On Wed, Mar 29, 2017 at 10:49 AM, Lars Volker wrote: >> Cool, thanks. My change has already been published to the S3 bucket, but I >> hadn't updated the

Re: Kudu version mismatch between impala-config.sh and toolchain

2017-03-29 Thread Matthew Jacobs
> On Wed, Mar 29, 2017 at 7:21 PM, Matthew Jacobs wrote: > >> Lars, I'm starting a new toolchain build now with that change as well >> as your latest change: >> >> * 5364e21 - (upstream/master, gerrit/master) IMPALA-4226, IMPALA-4227: >> bump max threads, han

Re: Kudu version mismatch between impala-config.sh and toolchain

2017-03-29 Thread Matthew Jacobs
Lars, I'm starting a new toolchain build now with that change as well as your latest change: * 5364e21 - (upstream/master, gerrit/master) IMPALA-4226, IMPALA-4227: bump max threads, handle dwz compressed symbols (2 days ago) On Wed, Mar 29, 2017 at 9:52 AM, Matthew Jacobs wrote: >

Re: Kudu version mismatch between impala-config.sh and toolchain

2017-03-29 Thread Matthew Jacobs
I'll get that version in to the toolchain repo: https://gerrit.cloudera.org/#/c/6509/ On Wed, Mar 29, 2017 at 5:52 AM, Lars Volker wrote: > Hi All, > > In impala-config.sh we consume Kudu version 16dd6e4. > > bin/impala-config.sh:124:export IMPALA_KUDU_VERSION=16dd6e4 > > The toolchain does not b

Re: Min/Max runtime filtering on Impala-Kudu

2017-03-27 Thread Matthew Jacobs
Thanks for writing this up, Sailesh. It sounds reasonable. On Mon, Mar 27, 2017 at 2:24 PM, Sailesh Mukil wrote: > On Mon, Mar 27, 2017 at 11:49 AM, Marcel Kornacker > wrote: > >> On Mon, Mar 27, 2017 at 11:42 AM, Sailesh Mukil >> wrote: >> > I will be working on a patch to add min/max filter s

Re: Toolchain - versioning dependencies with the same version number

2017-02-27 Thread Matthew Jacobs
Just catching up to this e-mail, though I had seen your code reviews and I think this approach makes sense. An additional concern would be how to identify how a toolchain package was built, and AFAIK this is tricky now if only the 'toolchain ID' is known. Before I saw this e-mail I was thinking abo

[Toolchain-CR] Add a script to build Kudu using existing toolchain artifacts

2017-02-27 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has abandoned this change. Change subject: Add a script to build Kudu using existing toolchain artifacts .. Abandoned moving to the new project -- To view, visit http://gerrit.cloudera.org:8080/6014 To

[Toolchain-CR] IMPALA-4983: Compile LZ4 in release mode

2017-02-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4983: Compile LZ4 in release mode .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6145 To unsubscribe, visit http

[Toolchain-CR] Add missing word to comment

2017-02-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Add missing word to comment .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6142 To unsubscribe, visit http

[Toolchain-CR] IMPALA-4966: Add flatbuffers 1.6.0 to toolchain

2017-02-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: IMPALA-4966: Add flatbuffers 1.6.0 to toolchain .. IMPALA-4966: Add flatbuffers 1.6.0 to toolchain Testing: Passed private verify-impala-toolchain

[Toolchain-CR] IMPALA-4966: Add flatbuffers 1.6.0 to toolchain

2017-02-24 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4966: Add flatbuffers 1.6.0 to toolchain .. Patch Set 2: gah, this wasn't submitted so gerrit/master and upstream/master are out of sync now (we

[Toolchain-CR] Update Kudu version to 2b0edbe

2017-02-23 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: Update Kudu version to 2b0edbe .. Update Kudu version to 2b0edbe Change-Id: I175133b1841449e311d637d10efc9d503612a3cf --- M buildall.sh 1 file changed

[Toolchain-CR] Update Kudu version to 2b0edbe

2017-02-23 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Update Kudu version to 2b0edbe .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6110 To unsubscribe, visit http

New Impala committer: Thomas Marshall

2017-02-23 Thread Matthew Jacobs
The Podling Project Management Committee (PPMC) for Apache Impala (incubating) has invited Thomas Marshall to become a committer and we are pleased to announce that he has accepted. Congratulations and welcome, Thomas!

[Toolchain-CR] Update Kudu version to 2b0edbe

2017-02-22 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Update Kudu version to 2b0edbe .. Patch Set 1: Replaces b73a714 because 2b0edbe is actually what's being used on Impala master right now. -- To view,

[Toolchain-CR] Update Kudu version to 2b0edbe

2017-02-22 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/6110 Change subject: Update Kudu version to 2b0edbe .. Update Kudu version to 2b0edbe Change-Id: I175133b1841449e311d637d10efc9d503612a3cf

Re: Cheap e2e test timing to find slowest tests (strategy = exhaustive)

2017-02-21 Thread Matthew Jacobs
e. Filed https://issues.cloudera.org/browse/IMPALA-4960 on myself On Tue, Feb 21, 2017 at 1:20 PM, Matthew Jacobs wrote: > Thanks David, this is interesting. > > I'll put up a patch to remove some of the tested file formats for > analytic functions since it shouldn't reall

Re: Cheap e2e test timing to find slowest tests (strategy = exhaustive)

2017-02-21 Thread Matthew Jacobs
Thanks David, this is interesting. I'll put up a patch to remove some of the tested file formats for analytic functions since it shouldn't really matter too much, unless there are differences in timing producing rows to the analytic functions, but I'm not sure that's the right way to get that kind

[Toolchain-CR] Bump Kudu version to b73a714

2017-02-17 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: Bump Kudu version to b73a714 .. Bump Kudu version to b73a714 Change-Id: I11e7533c0b50ef67fa53eb55096b91c7d39dec10 --- M buildall.sh 1 file changed, 2

[Toolchain-CR] Bump Kudu version to b73a714

2017-02-17 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Bump Kudu version to b73a714 .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6057 To unsubscribe, visit http

[Toolchain-CR] Bump Kudu version to b73a714

2017-02-17 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Bump Kudu version to b73a714 .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6057/1/buildall.sh File buildall.sh: PS1, Line 262: 0.8.0-RC1

[Toolchain-CR] Bump Kudu version to b73a714

2017-02-17 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/6057 Change subject: Bump Kudu version to b73a714 .. Bump Kudu version to b73a714 Change-Id: I11e7533c0b50ef67fa53eb55096b91c7d39dec10

[Toolchain-CR] Add a script to build Kudu using existing toolchain artifacts

2017-02-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/6014 Change subject: Add a script to build Kudu using existing toolchain artifacts .. Add a script to build Kudu using existing toolchain

Re: Can a committer carry a +2 and submit a gvm?

2017-02-14 Thread Matthew Jacobs
done On Tue, Feb 14, 2017 at 9:54 AM, David Knupp wrote: > Sorry, sent to wrong mailing list initially. > >> *From:* David Knupp >> *Date:* February 14, 2017 at 9:52 AM >> *To:* impala-team >> *Subject:* Can a committer carry a +2 and submit a gvm? >> >> Just rebased: >> https://gerrit.cloudera

[Toolchain-CR] Bump Kudu version to latest master (cd7b0dd)

2017-02-08 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Bump Kudu version to latest master (cd7b0dd) .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5928 To unsubscribe, visit http

[Toolchain-CR] Bump Kudu version to latest master (cd7b0dd)

2017-02-08 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: Bump Kudu version to latest master (cd7b0dd) .. Bump Kudu version to latest master (cd7b0dd) Kudu's cmake no longer needs to be patched to work a

[Toolchain-CR] Bump Kudu version to latest master (cd7b0dd)

2017-02-07 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/5928 Change subject: Bump Kudu version to latest master (cd7b0dd) .. Bump Kudu version to latest master (cd7b0dd) Kudu's cmake no l

[Toolchain-CR] IMPALA-4477: Bump Kudu to latest commit on master (e018a83)

2016-12-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: IMPALA-4477: Bump Kudu to latest commit on master (e018a83) .. IMPALA-4477: Bump Kudu to latest commit on master (e018a83

[Toolchain-CR] IMPALA-4477: Bump Kudu to latest commit on master (e018a83)

2016-12-15 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4477: Bump Kudu to latest commit on master (e018a83) .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5460 To unsubscribe

Need a +2 to revert the revert to bump the kudu version

2016-12-13 Thread Matthew Jacobs
I submitted a gvo optimistically (passed) but this still needs a +2: https://gerrit.cloudera.org/#/c/5487/ The fix that was needed for this to pass tests is: commit 73e41cea196703701d40cc67f919287fb3511b9b Author: Matthew Jacobs Date: Mon Dec 12 14:32:07 2016 -0800 IMPALA-4642: Fix

Re: Overriding impala-config.sh values

2016-12-12 Thread Matthew Jacobs
+1 Obviously there's a lot of pain around the team with the toolchain version bumping. On Sat, Dec 10, 2016 at 8:16 AM, Lars Volker wrote: > +1 > > On Thu, Dec 8, 2016 at 1:36 PM, Jim Apple wrote: > >> I like this idea. >> >> On Thu, Dec 8, 2016 at 1:26 PM, Tim Armstrong >> wrote: >> > Hi All,

Re: Important: new shell & bootstrap required after rebasing, again

2016-12-12 Thread Matthew Jacobs
wing it out in case someone has an > idea off the top of their heads as to what the issue might be > >> Matthew Jacobs <mailto:m...@cloudera.com> >> December 11, 2016 at 9:07 AM >> Try just setting the toolchain build id manually: >> export IMPALA_TOOLCHAIN_BUILD_ID

Re: Important: new shell & bootstrap required after rebasing, again

2016-12-11 Thread Matthew Jacobs
Try just setting the toolchain build id manually: export IMPALA_TOOLCHAIN_BUILD_ID=308-96a4cc516e Anyway this is annoying, +1 to Tim's proposal for improved environment configs. On Sun, Dec 11, 2016 at 9:04 AM, Matthew Jacobs wrote: > The URL is still referencing an old toolchain ver

Re: Important: new shell & bootstrap required after rebasing, again

2016-12-11 Thread Matthew Jacobs
7.83 > Connecting to native-toolchain.s3.amazonaws.com > (native-toolchain.s3.amazonaws.com)|54.231.237.83|:443... connected. > HTTP request sent, awaiting response... 403 Forbidden > 2016-12-11 08:56:17 ERROR 403: Forbidden. > > On Sat, Dec 10, 2016 at 11:11 PM, Matthew Jaco

Important: new shell & bootstrap required after rebasing, again

2016-12-10 Thread Matthew Jacobs
Hi all, We had to bump the Kudu version again so unfortunately you'll have to refresh your environment again to get the latest Kudu. After fetching the latest Impala commits (including commit 39017adf) and rebasing, you'll need to: 1) start a new shell OR make sure to export IMPALA_TOOLCHAIN_BU

[Toolchain-CR] IMPALA-4477: Bump Kudu to latest commit on master (e018a83)

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#2). Change subject: IMPALA-4477: Bump Kudu to latest commit on master (e018a83) .. IMPALA-4477: Bump Kudu to latest commit on master (e018a83

[Toolchain-CR] IMPALA-4477: Bump Kudu to latest commit on master (e018a83)

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4477: Bump Kudu to latest commit on master (e018a83) .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5460/1/buildall.sh File

[Toolchain-CR] IMPALA-4477: Bump Kudu to latest commit on master (e018a83)

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4477: Bump Kudu to latest commit on master (e018a83) .. Patch Set 1: building now -- To view, visit http://gerrit.cloudera.org:8080/5460 To

[Toolchain-CR] IMPALA-4636: Add support for SLES12 for Impala/Kudu integration

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: IMPALA-4636: Add support for SLES12 for Impala/Kudu integration .. IMPALA-4636: Add support for SLES12 for Impala/Kudu integration Adds sles12 support

[Toolchain-CR] IMPALA-4477: Bump Kudu to latest commit on master (e018a83)

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/5460 Change subject: IMPALA-4477: Bump Kudu to latest commit on master (e018a83) .. IMPALA-4477: Bump Kudu to latest commit on master

[Toolchain-CR] IMPALA-4636: Add support for SLES12 for Impala/Kudu integration

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4636: Add support for SLES12 for Impala/Kudu integration .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5454 To

[Toolchain-CR] IMPALA-4636: Add support for SLES12 for Impala/Kudu integration

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4636: Add support for SLES12 for Impala/Kudu integration .. Patch Set 3: Code-Review+2 carrying the +2 -- To view, visit http://gerrit.cloudera.org

[Toolchain-CR] IMPALA-4636: Add support for SLES12 for Impala/Kudu integration

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4636: Add support for SLES12 for Impala/Kudu integration .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5454/2/source/kudu/build.sh

[Toolchain-CR] IMPALA-4636: Add support for SLES12 for Impala/Kudu integration

2016-12-09 Thread Matthew Jacobs (Code Review)
54 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I87d22028e08b19a1b7f39cb5def115a82560e292 Gerrit-PatchSet: 3 Gerrit-Project: Toolchain Gerrit-Branch: master Gerrit-Owner: Matthew Jacobs Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Ma

[Toolchain-CR] IMPALA-4636: Add support for SLES12 for Impala/Kudu integration

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#2). Change subject: IMPALA-4636: Add support for SLES12 for Impala/Kudu integration .. IMPALA-4636: Add support for SLES12 for Impala/Kudu integration Adds sles12 support for

[Toolchain-CR] IMPALA-4636: Add support for SLES12 for Impala/Kudu integration

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4636: Add support for SLES12 for Impala/Kudu integration .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5454/1/source/kudu/build.sh

[Toolchain-CR] IMPALA-4636: Add support for SLES12 for Impala/Kudu integration

2016-12-09 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/5454 Change subject: IMPALA-4636: Add support for SLES12 for Impala/Kudu integration .. IMPALA-4636: Add support for SLES12 for Impala/Kudu

[Toolchain-CR] Allow overriding of Kudu github URL

2016-12-08 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Allow overriding of Kudu github URL .. Patch Set 1: Code-Review+2 (1 comment) thanks http://gerrit.cloudera.org:8080/#/c/5430/1//COMMIT_MSG Commit Message

[Toolchain-CR] Bump kudu version to a more recent commit on master

2016-12-07 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Bump kudu version to a more recent commit on master .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5411 To unsubscribe, visit

Re: Important: new shell & bootstrap required after rebasing

2016-12-07 Thread Matthew Jacobs
gt; https://native-toolchain.s3.amazonaws.com/build/301-05e1fe2b3b/kudu/60aa54e-gcc-4.9.2/kudu-60aa54e-gcc-4.9.2-ec2-package-ubuntu-14-04.tar.gz > > On Tue, Dec 6, 2016 at 9:48 PM, Matthew Jacobs wrote: >> The Kudu version was updated to the latest master. After fetching the >> lates

Important: new shell & bootstrap required after rebasing

2016-12-06 Thread Matthew Jacobs
The Kudu version was updated to the latest master. After fetching the latest Impala commits (including commit 5188f87) and rebasing, you'll need to: 1) start a new shell OR make sure to export IMPALA_TOOLCHAIN_BUILD_ID=301-05e1fe2b3b 2) Run buildall.sh, that will: a) download the new Kudu bit

[Toolchain-CR] IMPALA-4477: Bump Kudu to latest commit on master (60aa54e)

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: IMPALA-4477: Bump Kudu to latest commit on master (60aa54e) .. IMPALA-4477: Bump Kudu to latest commit on master (60aa54e) Update to Kudu git hash

[Toolchain-CR] IMPALA-4477: Bump Kudu to latest commit on master (60aa54e)

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4477: Bump Kudu to latest commit on master (60aa54e) .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5370 To unsubscribe

[Toolchain-CR] IMPALA-4477: Bump Kudu to latest commit on master (60aa54e)

2016-12-05 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/5370 Change subject: IMPALA-4477: Bump Kudu to latest commit on master (60aa54e) .. IMPALA-4477: Bump Kudu to latest commit on master

[Toolchain-CR] Add CLEAN AFTER BUILD option to clean files during build

2016-12-05 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Add CLEAN_AFTER_BUILD option to clean files during build .. Patch Set 2: Code-Review+2 nope -- To view, visit http://gerrit.cloudera.org:8080/5313 To

[Toolchain-CR] Add CLEAN AFTER BUILD option to clean files during build

2016-12-01 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Add CLEAN_AFTER_BUILD option to clean files during build .. Patch Set 1: (2 comments) Thanks! http://gerrit.cloudera.org:8080/#/c/5313/1/init.sh File init.sh

Re: Adding some guard rails to Kudu

2016-11-30 Thread Matthew Jacobs
creating Kudu table names as > '::'. > > > On Wed, Nov 30, 2016 at 4:03 PM, Todd Lipcon wrote: > >> On Wed, Nov 30, 2016 at 3:52 PM, Matthew Jacobs wrote: >> >>> 1) I think that makes sense, though we need to know what the error >>> conditions

Re: Adding some guard rails to Kudu

2016-11-30 Thread Matthew Jacobs
On Wed, Nov 30, 2016 at 4:03 PM, Todd Lipcon wrote: > On Wed, Nov 30, 2016 at 3:52 PM, Matthew Jacobs wrote: > >> 1) I think that makes sense, though we need to know what the error >> conditions are, when those errors occur (e.g. at table creation, add >> column, writing

Re: Adding some guard rails to Kudu

2016-11-30 Thread Matthew Jacobs
1) I think that makes sense, though we need to know what the error conditions are, when those errors occur (e.g. at table creation, add column, writing data) and need tests to validate the expected negative cases. I can certainly guess, though I'd like for the affected API calls w/ new expected beh

[Toolchain-CR] Fix build failures from git clean

2016-11-30 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Fix build failures from git clean .. Patch Set 1: Code-Review+2 thanks! -- To view, visit http://gerrit.cloudera.org:8080/5285 To unsubscribe, visit http

[Toolchain-CR] IMPALA-4477: Bump Kudu version to latest commit on master

2016-11-17 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#3). Change subject: IMPALA-4477: Bump Kudu version to latest commit on master .. IMPALA-4477: Bump Kudu version to latest commit on master Kudu commit

Re: Kudu start error with low ntpdate "maximum error"

2016-11-16 Thread Matthew Jacobs
what is going on, and if I > hit this again I'll re-open the thread. For now, I am not planning to > submit a patch to "fix" it because I'm not sure increasing the number > is the real solution. > > On Wed, Nov 16, 2016 at 11:00 AM, Matthew Jac

Re: Kudu start error with low ntpdate "maximum error"

2016-11-16 Thread Matthew Jacobs
e error isn't a good idea, and we'll need to figure out something else. On Wed, Nov 16, 2016 at 10:55 AM, Jim Apple wrote: > How do we bump up the allowable error? > > On Wed, Nov 16, 2016 at 10:20 AM, Matthew Jacobs wrote: >> I asked on the Kudu slack channel, they have

Re: Kudu start error with low ntpdate "maximum error"

2016-11-16 Thread Matthew Jacobs
zone while most > Impala developers work in America/Los_Angeles. > > On Wed, Nov 16, 2016 at 9:10 AM, Matthew Jacobs wrote: >> No problem. If this happens again we should ask the Kudu developers. I >> haven't seen this before - I wonder if it could be some weirdness on >>

[Toolchain-CR] IMPALA-4477: Bump Kudu version to latest commit on master

2016-11-16 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#2). Change subject: IMPALA-4477: Bump Kudu version to latest commit on master .. IMPALA-4477: Bump Kudu version to latest commit on master Kudu commit

compile_commands issues after using distcc?

2016-11-16 Thread Matthew Jacobs
Hi all, Do any vim/YouCompleteMe users (or anyone else using the compile_commands.json) have issues after enabling distcc? I've found my vim completion (via YouCompleteMe) no longer works. I'm curious to know if anyone has any workarounds or suggestions. Thanks

Re: Kudu start error with low ntpdate "maximum error"

2016-11-16 Thread Matthew Jacobs
8b023-gcc-4.9.2-ec2-package-ubuntu-14-04.tar.gz";. > I'll add "ps aux | grep kudu" to the logging this machine does on > error, so we'll have it next time, but I did "ps -Afly" on exit and > there were no kudu processes running, it looks like. > &

  1   2   3   4   5   6   7   >