Mailing-List: contact reviews-help@impala.incubator.apache.org; run by ezmlm
Precedence: bulk
Message-Id: <201612010802.uB182F7Q018997@ip-10-146-233-104.ec2.internal>
Date: Thu, 1 Dec 2016 08:02:15 +0000
From: "Dan Hecht (Code Review)" <gerrit@cloudera.org>
To: Matthew Jacobs <mj@cloudera.com>, impala-cr@cloudera.com,
        reviews@impala.incubator.apache.org
CC: David Ribeiro Alves <dralves@apache.org>
Reply-To: dhecht@cloudera.com
Subject: =?UTF-8?Q?=5BImpala-ASF-CR=5D_IMPALA-3788=3A_Add_flag_for_Kudu_read-your-writes=0A?=
In-Reply-To: <gerrit.1480547124000.I003aba410548bc9158d1e11abbdcf710c31a82ff@gerrit.cloudera.org>
References: <gerrit.1480547124000.I003aba410548bc9158d1e11abbdcf710c31a82ff@gerrit.cloudera.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Content-Disposition: inline
User-Agent: Gerrit/2.12.2
archived-at: Thu, 01 Dec 2016 08:02:20 -0000

Dan Hecht has posted comments on this change.

Change subject: IMPALA-3788: Add flag for Kudu read-your-writes
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/5288/1/be/src/exec/kudu-scanner.cc
File be/src/exec/kudu-scanner.cc:

Line 139:         kudu::client::KuduScanner::READ_AT_SNAPSHOT), "Could not set scanner ReadMode.");
rather than relying on the kudu default, do you think we should always call SetReadMode() with the appropriate flag depending on the Impala setting? Seems more explicit that way.

Also, if we plan to make this the default soon, do you think we should make the option have the opposite polarity (i.e. read_latest -- so the default becomes 'false' in the long term)? Alternatively, is there any chance there would be more than two values for the read mode in the future? In that case, maybe the option should be --kudu_read_mode={snapshot, latest}.


-- 
To view, visit http://gerrit.cloudera.org:8080/5288
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I003aba410548bc9158d1e11abbdcf710c31a82ff
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Matthew Jacobs <mj@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhecht@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dralves@apache.org>
Gerrit-Reviewer: Matthew Jacobs <mj@cloudera.com>
Gerrit-HasComments: Yes