Update tableby.go

table: add QueryBy and DeleteBy for multi-column filtering at Table level

Provide user-friendly methods QueryBy and DeleteBy that apply multi-column (col→val) filters across all buckets.
These new methods panic if given nil or empty filters, ensuring predictable behavior and avoiding accidental full-table operations.
QueryBy returns nil when no matches are found, consistent with the low-level API.
DeleteBy performs removals without returning deleted counts, simplifying the interface for typical use cases.

Author: neurlang

tableby.go

@@ -8,8 +8,7 @@ func (t *Table) QueryBy(filters map[int]string) [][]string {
 		panic("QueryBy: filters must not be nil or empty")
 	}
 	var result [][]string
-	// delegate to each bucket
-	for _, buck := range t.buckets {
+	for _, buck := range t.b {
 		rows := buck.getBy(filters)
 		if rows != nil {
 			result = append(result, rows...)
@@ -23,23 +22,12 @@ func (t *Table) QueryBy(filters map[int]string) [][]string {
 
 // DeleteBy deletes all rows matching every (col→val).
 // Panics if filters is nil or empty.
-// Returns the total number of rows deleted.
-func (t *Table) DeleteBy(filters map[int]string) int {
+func (t *Table) DeleteBy(filters map[int]string) {
 	if filters == nil || len(filters) == 0 {
 		panic("DeleteBy: filters must not be nil or empty")
 	}
-	deleted := 0
-	for i := range t.buckets {
-		// count before
-		before := len(t.buckets[i].getBy(filters))
-		t.buckets[i].removeBy(filters)
-		// count after
-		afterRows := t.buckets[i].getBy(filters)
-		if afterRows == nil {
-			deleted += before
-		} else {
-			deleted += before - len(afterRows)
-		}
+
+	for i := range t.b {
+		t.b[i].removeBy(filters)
 	}
-	return deleted
 }