Merge pull request #63 from ydb-platform/result-set

moved HasNextResultSet from unary result interface to common result i…

Author: asmyasnikov

CHANGELOG.md

@@ -17,6 +17,8 @@
 * Added `DoTx()` API method into `table.Client`
 * Added `String()` method into `ConnectParams` for serialize params to connection string
 * Added early exit from Rollback for committed transaction
+* Moved `HasNextResultSet()` method from `Result` interface to common `result` interface.
+  It provides access to `HasNextResultSet()` on both result interfaces (unary and stream results)
 
 ## 3.4.4
 * Prefer `ydb.table.types.Scanner` scanner implementation over `sql.Scanner`, when both available.

table/result/result.go

@@ -29,6 +29,12 @@ import (
 // After that, NextResultSet(), NextRow() will return false.
 type result interface {
 
+	// HasNextResultSet reports whether result set may be advanced.
+	// It may be useful to call HasNextResultSet() instead of NextResultSet() to look ahead
+	// without advancing the result set.
+	// Note that it does not work with sets from stream.
+	HasNextResultSet() bool
+
 	// NextResultSet selects next result set in the result.
 	// columns - names of columns in the resultSet that will be scanned
 	// It returns false if there are no more result sets.
@@ -99,19 +105,9 @@ type result interface {
 type Result interface {
 	result
 
-	// HasNextResultSet reports whether result set may be advanced.
-	// It may be useful to call HasNextResultSet() instead of NextResultSet() to look ahead
-	// without advancing the result set.
-	// Note that it does not work with sets from stream.
-	HasNextResultSet() bool
-
 	// ResultSetCount returns number of result sets.
 	// Note that it does not work if r is the result of streaming operation.
 	ResultSetCount() int
-
-	// TotalRowCount returns the number of rows among the all result sets.
-	// Note that it does not work if r is the result of streaming operation.
-	TotalRowCount() int
 }
 
 type StreamResult interface {

trace/table.go

@@ -66,9 +66,7 @@ type (
 	}
 	result interface {
 		resultErr
-
 		ResultSetCount() int
-		TotalRowCount() int
 	}
 	SessionNewStartInfo struct {
 		// Context make available context in trace callback function.