[impala] branch master updated: IMPALA-9835: make kudu_scan_token binary

[ta...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

tarmstrong pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git


The following commit(s) were added to refs/heads/master by this push:
     new 51687e5  IMPALA-9835: make kudu_scan_token binary
51687e5 is described below

commit 51687e54265aa833453b8574cf21fe615e3da556
Author: Tim Armstrong <ta...@cloudera.com>
AuthorDate: Fri Jun 5 15:35:44 2020 -0700

    IMPALA-9835: make kudu_scan_token binary
    
    We got log spam from protobufs because the scan
    tokens are not valid UTF-8 (and not intended to
    be).
    
    Change-Id: I2db9e4aad777065f6bf6ab1c9add3c99e8a0cf0a
    Reviewed-on: http://gerrit.cloudera.org:8080/16040
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
    Reviewed-by: Thomas Tauber-Marshall <tm...@cloudera.com>
---
 common/protobuf/planner.proto | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/common/protobuf/planner.proto b/common/protobuf/planner.proto
index a22ff9b..80cf305 100644
--- a/common/protobuf/planner.proto
+++ b/common/protobuf/planner.proto
@@ -72,5 +72,5 @@ message ScanRangePB {
   // One of these must be set for every ScanRangePB.
   optional HdfsFileSplitPB hdfs_file_split = 1;
   optional HBaseKeyRangePB hbase_key_range = 2;
-  optional string kudu_scan_token = 3;
+  optional bytes kudu_scan_token = 3;
 }