[1/2] impala git commit: IMPALA-4671: (part-2) Replace kudu::ServicePool with one that uses Impala threads

2017-12-07 Thread tarmstrong
Repository: impala Updated Branches: refs/heads/master a94d6068c -> f3fa3e017 IMPALA-4671: (part-2) Replace kudu::ServicePool with one that uses Impala threads The KuduRPC subsystem uses kudu::ServicePool to service all incoming RPCs. Since this lives inside the Kudu codebase, all instrumenta

[2/2] impala git commit: IMPALA-6081: Fix test_basic_filters runtime profile failure

2017-12-07 Thread tarmstrong
IMPALA-6081: Fix test_basic_filters runtime profile failure test_basic_filters has been occasionally failing due to a line missing from a runtime profile for a particular query. The problem is that the query returns all of its results before all of its fragment instances are finished executing (d

[3/4] impala git commit: IMPALA-6281: Fix use-after-free in InitAuth()

2017-12-07 Thread kwho
IMPALA-6281: Fix use-after-free in InitAuth() Previously, we implicitly create a local string object created from the char* in argv[0] when calling InitAuth(). This string object goes out of scope once InitAuth() returns but the pointer of this local string's buffer is passed to the Sasl library w

[4/4] impala git commit: IMPALA-4671: (part-1) Copy kudu::ServicePool into Impala namespace

2017-12-07 Thread kwho
IMPALA-4671: (part-1) Copy kudu::ServicePool into Impala namespace *** THIS PATCH IS NOT FOR REVIEW. IT IS JUST UPLOADED TO MAKE THE REVIEW OF PART-2 EASIER IN GERRIT BY SHOWING ONLY THE DIFFS OF THAT PATCH AGAINST THIS PATCH. MORE EXPLANATION BELOW*** The KuduRPC subsystem uses kudu::ServicePool

[1/4] impala git commit: IMPALA-3804: Re-enable per-scan filtering for sequence-based scanners

2017-12-07 Thread kwho
Repository: impala Updated Branches: refs/heads/master ae6ecf51e -> a94d6068c IMPALA-3804: Re-enable per-scan filtering for sequence-based scanners IMPALA-3798 disabled per-scan filtering for sequence- based scanners due to a race between runtime filter arrival and header splits processing. T

[2/4] impala git commit: IMPALA-6262: Always initialize runtime profile for DataSink

2017-12-07 Thread kwho
IMPALA-6262: Always initialize runtime profile for DataSink This change moves the creation of the runtime profile from DataSink::Prepare() to the ctor of DataSink derived classes. This makes sure that DataSink::Close() and other functions can access the profile even if the DataSink fails to initi