change functionName finalize to close

apache · morningman · Jun 22, 2020 · Jun 1, 2020 · Jun 1, 2020 · Jun 4, 2020
commit 32770821db2c5912dfcefab5e2c7707a495c3dc2
diff --git a/be/src/olap/push_handler.cpp b/be/src/olap/push_handler.cpp
@@ -522,7 +522,7 @@ OLAPStatus PushHandler::_convert(TabletSharedPtr cur_tablet,
                }
            }

-            reader->finalize();
+            reader->close();

            if (!reader->validate_checksum()) {
                LOG(WARNING) << "pushed delta file has wrong checksum.";

diff --git a/be/src/olap/push_handler.h b/be/src/olap/push_handler.h
@@ -229,7 +229,7 @@ class PushBrokerReader {
    OLAPStatus next(ContiguousRow* row);
    void print_profile();

-    OLAPStatus finalize() {
+    OLAPStatus close() {
        _ready = false;
        return OLAP_SUCCESS;
    }
-Original file line number
+Diff line change
@@ -522,7 +522,7 @@ OLAPStatus PushHandler::_convert(TabletSharedPtr cur_tablet,
                     }
                 }
-                reader->finalize();
+                reader->close();
                   xy720 marked this conversation as resolved.
              

          
            Show resolved
            Hide resolved
        

                
  
            
  

  
    
  
    
      

              Uh oh!

              
There was an error while loading. Please reload this page.
                 if (!reader->validate_checksum()) {
                     LOG(WARNING) << "pushed delta file has wrong checksum.";
-          Expand Down