Merge pull request #66 from gcpug/fix-bug

Fix boundary test

Author: kazegusuri

server/server.go

@@ -867,7 +867,7 @@ func sendResult(stream spannerpb.Spanner_StreamingReadServer, tx *transaction, i
 			// // But set the first response for now.
 			// stats = nil
 
-			values = values[:]
+			values = values[:0]
 		}
 		return nil
 	})

server/server_test.go

@@ -20,6 +20,7 @@ import (
 	"regexp"
 	"sort"
 	"strconv"
+	"strings"
 	"testing"
 	"time"
 
@@ -899,6 +900,66 @@ func TestExecuteStreamingSql_Success(t *testing.T) {
 				{makeStringValue("3"), makeStringValue("bbb"), makeStringValue("3")},
 			},
 		},
+		"CompositePrimaryKeys_Condition2": {
+			sql: fmt.Sprintf(`SELECT %s FROM CompositePrimaryKeys`,
+				strings.Join(strings.Split(strings.Repeat("XYZ", 10), ""), ", ")),
+			expected: [][]*structpb.Value{
+				{
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y2"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y2"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y2"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y2"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y2"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y2"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y2"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y2"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y2"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y2"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y3"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y3"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y3"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y3"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y3"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y3"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y3"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y3"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y3"), makeStringValue("z"),
+					makeStringValue("x1"), makeStringValue("y3"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y4"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y4"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y4"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y4"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y4"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y4"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y4"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y4"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y4"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y4"), makeStringValue("z"),
+				},
+				{
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+					makeStringValue("x2"), makeStringValue("y5"), makeStringValue("z"),
+				},
+			},
+		},
 		"ArrayOfStruct": {
 			sql: `SELECT ARRAY(SELECT STRUCT<Id int64, Value string>(1,"xx") x)`,
 			fields: []*spannerpb.StructType_Field{