From 40d80515a18c3c249cc4df62ba14243127236fa6 Mon Sep 17 00:00:00 2001
From: Wen Sun <iwendellsun@gmail.com>
Date: Thu, 16 Mar 2023 21:25:47 +0900
Subject: [PATCH] Use the GetLoadingProgress api instead of the
 InMemoryPercentages in the ShowPartition api response

Fix load collection hang issue

Signed-off-by: Wen Sun <iwendellsun@gmail.com>
---
 client/client_grpc_collection.go |  8 +++----
 client/client_grpc_partition.go  | 39 ++++----------------------------
 2 files changed, 8 insertions(+), 39 deletions(-)

diff --git a/client/client_grpc_collection.go b/client/client_grpc_collection.go
index 30be56e7f..86cf43641 100644
--- a/client/client_grpc_collection.go
+++ b/client/client_grpc_collection.go
@@ -373,15 +373,15 @@ func (c *GrpcClient) LoadCollection(ctx context.Context, collName string, async
 			default:
 			}
 
-			coll, err := c.ShowCollection(ctx, collName)
+			progress, err := c.GetLoadingProgress(ctx, collName, nil)
 			if err != nil {
 				return err
 			}
-			if coll.Loaded {
-				break
+			if progress == 100 {
+				return nil
 			}
 
-			time.Sleep(200 * time.Millisecond) // TODO change to configuration
+			time.Sleep(500 * time.Millisecond)
 		}
 	}
 	return nil
diff --git a/client/client_grpc_partition.go b/client/client_grpc_partition.go
index 87612331e..9bef5f984 100644
--- a/client/client_grpc_partition.go
+++ b/client/client_grpc_partition.go
@@ -149,23 +149,6 @@ func (c *GrpcClient) LoadPartitions(ctx context.Context, collName string, partit
 			return err
 		}
 	}
-	partitions, err := c.ShowPartitions(ctx, collName)
-	if err != nil {
-		return err
-	}
-	m := make(map[string]int64)
-	for _, partition := range partitions {
-		m[partition.Name] = partition.ID
-	}
-	// load partitions ids
-	ids := make(map[int64]struct{})
-	for _, partitionName := range partitionNames {
-		id, has := m[partitionName]
-		if !has {
-			return fmt.Errorf("Collection %s does not has partitions %s", collName, partitionName)
-		}
-		ids[id] = struct{}{}
-	}
 
 	req := &server.LoadPartitionsRequest{
 		DbName:         "", // reserved
@@ -187,28 +170,14 @@ func (c *GrpcClient) LoadPartitions(ctx context.Context, collName string, partit
 				return errors.New("context deadline exceeded")
 			default:
 			}
-			partitions, err := c.ShowPartitions(ctx, collName)
+			percentage, err := c.GetLoadingProgress(ctx, collName, partitionNames)
 			if err != nil {
 				return err
 			}
-			foundLoading := false
-			loaded := 0
-			for _, partition := range partitions {
-				if _, has := ids[partition.ID]; !has {
-					continue
-				}
-				if !partition.Loaded {
-					//Not loaded
-					foundLoading = true
-					break
-				}
-				loaded++
-			}
-			if foundLoading || loaded < len(partitionNames) {
-				time.Sleep(time.Millisecond * 100)
-				continue
+			if percentage != 100 {
+				return nil
 			}
-			break
+			time.Sleep(500 * time.Millisecond)
 		}
 	}