Update tableby.go

Add QueryBy wrapper to skip holes; rename original to QueryByHoles

This change clarifies the API behavior around holes caused by deleted rows.
The new `QueryBy` provides a user-friendly multi-clause lookup that
filters out nil or empty rows, ensuring consistent results for clients.
The original implementation is retained as `QueryByHoles` for advanced
use cases where raw bucket contents (including holes) are needed.
No changes were made to `DeleteBy` as deletion semantics remain unaffected.

Author: neurlang

tableby.go

@@ -1,33 +1,53 @@
 package table
 
-// QueryBy finds all rows matching every (col→val).
+// QueryByHoles finds all rows matching every (col→val), including any holes.
+// Panics if filters is nil or empty.
+// Returns nil for no matches.
+func (t *Table) QueryByHoles(filters map[int]string) [][]string {
+    if filters == nil || len(filters) == 0 {
+        panic("QueryByHoles: filters must not be nil or empty")
+    }
+    var result [][]string
+    for _, buck := range t.b {
+        rows := buck.getBy(filters)
+        if rows != nil {
+            result = append(result, rows...)
+        }
+    }
+    if len(result) == 0 {
+        return nil
+    }
+    return result
+}
+
+// QueryBy finds all rows matching every (col→val), skipping any holes.
 // Panics if filters is nil or empty.
 // Returns nil for no matches.
 func (t *Table) QueryBy(filters map[int]string) [][]string {
-	if filters == nil || len(filters) == 0 {
-		panic("QueryBy: filters must not be nil or empty")
-	}
-	var result [][]string
-	for _, buck := range t.b {
-		rows := buck.getBy(filters)
-		if rows != nil {
-			result = append(result, rows...)
-		}
-	}
-	if len(result) == 0 {
-		return nil
-	}
-	return result
+    raw := t.QueryByHoles(filters)
+    if raw == nil {
+        return nil
+    }
+    var filtered [][]string
+    for _, row := range raw {
+        if row != nil && len(row) > 0 {
+            filtered = append(filtered, row)
+        }
+    }
+    if len(filtered) == 0 {
+        return nil
+    }
+    return filtered
 }
 
 // DeleteBy deletes all rows matching every (col→val).
 // Panics if filters is nil or empty.
 func (t *Table) DeleteBy(filters map[int]string) {
-	if filters == nil || len(filters) == 0 {
-		panic("DeleteBy: filters must not be nil or empty")
-	}
+    if filters == nil || len(filters) == 0 {
+        panic("DeleteBy: filters must not be nil or empty")
+    }
 
-	for i := range t.b {
-		t.b[i].removeBy(filters)
-	}
+    for i := range t.b {
+        t.b[i].removeBy(filters)
+    }
 }