You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by gr...@apache.org on 2020/04/19 13:10:58 UTC

[kudu] branch master updated: tablet_service: init serializer with valid schema

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 27288e2  tablet_service: init serializer with valid schema
27288e2 is described below

commit 27288e243b8f51b7dd5f76729b57d825a753ae3b
Author: Andrew Wong <aw...@cloudera.com>
AuthorDate: Sat Apr 18 16:44:13 2020 -0700

    tablet_service: init serializer with valid schema
    
    Seems like the mergings of 163cd256ae and c7c4d47 were out of sync with
    each other, leading to a logical conflict that resulted in new scanners
    initializing with explicitly disallowed null projections.
    
    Change-Id: Ic0355db2b7b39b99df6aded876ad22f6d2fbd776
    Reviewed-on: http://gerrit.cloudera.org:8080/15754
    Tested-by: Kudu Jenkins
    Reviewed-by: Hao Hao <ha...@cloudera.com>
    Reviewed-by: Grant Henke <gr...@apache.org>
---
 src/kudu/tserver/tablet_service.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/kudu/tserver/tablet_service.cc b/src/kudu/tserver/tablet_service.cc
index 34eda3d..ae22983 100644
--- a/src/kudu/tserver/tablet_service.cc
+++ b/src/kudu/tserver/tablet_service.cc
@@ -2567,7 +2567,7 @@ Status TabletServiceImpl::HandleNewScanRequest(TabletReplica* replica,
 
   s = result_collector->InitSerializer(scan_pb.row_format_flags(),
                                        projection,
-                                       *scanner->client_projection_schema());
+                                       *client_projection);
   if (!s.ok()) {
     *error_code = TabletServerErrorPB::INVALID_SCAN_SPEC;
     return s;