fix Client behaviour for output_fields differs from documentation (#398)

Signed-off-by: ryjiang <[email protected]>
milvus-io · Dec 25, 2024 · 1750962 · 1750962
1 parent 6941413
commit 1750962
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/milvus/grpc/Data.ts b/milvus/grpc/Data.ts
@@ -977,6 +977,7 @@ export class Data extends Collection {
       'Query',
       {
         ...data,
+        output_fields: data.output_fields || ['*'],
         query_params: parseToKeyValue({ ...limits, ...offset }),
       },
       data.timeout || this.timeout

diff --git a/test/grpc/Data.spec.ts b/test/grpc/Data.spec.ts
@@ -573,6 +573,20 @@ describe(`Data.API`, () => {
     expect(res4.data.length).toBe(default_values.length);
   });
 
+  it(`Query withouth output fields should success`, async () => {
+    const res = await milvusClient.query({
+      collection_name: COLLECTION_NAME,
+      expr: 'id > 0',
+      offset: 0,
+      limit: 3,
+    });
+
+    expect(res.status.error_code).toEqual(ErrorCode.SUCCESS);
+    expect(res.data.length).toBe(3);
+    // fields of data should > 1
+    expect(Object.keys(res.data[0]).length).toBeGreaterThan(1);
+  });
+
   it(`Query with count(*)`, async () => {
     const queryString = 'count(*)';
     const res = await milvusClient.query({