Support hash functions in distributions. Add distribution key in inte…

…rfaces
pg-sharding · Feb 14, 2024 · 6d3a752 · 6d3a752
1 parent ecddbee
commit 6d3a752
Show file tree

Hide file tree

Showing 14 changed files with 1,016 additions and 846 deletions.
diff --git a/pkg/meta/meta.go b/pkg/meta/meta.go
@@ -206,10 +206,8 @@ func processAlter(ctx context.Context, astmt spqrparser.Statement, mngr EntityMg
 func processAlterDistribution(ctx context.Context, astmt spqrparser.Statement, mngr EntityMgr, cli *clientinteractor.PSQLInteractor) error {
 	switch stmt := astmt.(type) {
 	case *spqrparser.AttachRelation:
-		rels := []*distributions.DistributedRelation{{
-			Name:        stmt.Relation.Name,
-			ColumnNames: stmt.Relation.Columns,
-		},
+		rels := []*distributions.DistributedRelation{
+			distributions.DistributedRelationFromSQL(stmt.Relation),
 		}
 		if err := mngr.AlterDistributionAttach(ctx, stmt.Distribution.ID, rels); err != nil {
 			return err

diff --git a/pkg/models/distributions/distribution.go b/pkg/models/distributions/distribution.go
@@ -3,39 +3,92 @@ package distributions
 import (
 	proto "github.com/pg-sharding/spqr/pkg/protos"
 	"github.com/pg-sharding/spqr/qdb"
+	spqrparser "github.com/pg-sharding/spqr/yacc/console"
 )
 
+type DistributionKeyEntry struct {
+	Column       string
+	HashFunction string
+}
+
 type DistributedRelation struct {
-	Name        string
-	ColumnNames []string
+	Name            string
+	DistributionKey []DistributionKeyEntry
 }
 
-func DistributedRelationFromDB(distr *qdb.DistributedRelation) *DistributedRelation {
-	return &DistributedRelation{
-		Name:        distr.Name,
-		ColumnNames: distr.ColumnNames,
+func DistributedRelationFromDB(rel *qdb.DistributedRelation) *DistributedRelation {
+	rdistr := &DistributedRelation{
+		Name: rel.Name,
 	}
+
+	for _, e := range rel.DistributionKey {
+		rdistr.DistributionKey = append(rdistr.DistributionKey, DistributionKeyEntry{
+			Column:       e.Column,
+			HashFunction: e.HashFunction,
+		})
+	}
+
+	return rdistr
 }
 
-func DistributedRelationToDB(distr *DistributedRelation) *qdb.DistributedRelation {
-	return &qdb.DistributedRelation{
-		Name:        distr.Name,
-		ColumnNames: distr.ColumnNames,
+func DistributedRelationToDB(rel *DistributedRelation) *qdb.DistributedRelation {
+	rdistr := &qdb.DistributedRelation{
+		Name: rel.Name,
 	}
+
+	for _, e := range rel.DistributionKey {
+		rdistr.DistributionKey = append(rdistr.DistributionKey, qdb.DistributionKeyEntry{
+			Column:       e.Column,
+			HashFunction: e.HashFunction,
+		})
+	}
+
+	return rdistr
 }
 
-func DistributedRelatitonToProto(distr *DistributedRelation) *proto.DistributedRelation {
-	return &proto.DistributedRelation{
-		Name:    distr.Name,
-		Columns: distr.ColumnNames,
+func DistributedRelatitonToProto(rel *DistributedRelation) *proto.DistributedRelation {
+	rdistr := &proto.DistributedRelation{
+		Name: rel.Name,
+	}
+
+	for _, e := range rel.DistributionKey {
+		rdistr.DistributionKey = append(rdistr.DistributionKey, &proto.DistributionKeyEntry{
+			Column:       e.Column,
+			HashFunction: e.HashFunction,
+		})
 	}
+
+	return rdistr
 }
 
 func DistributedRelationFromProto(rel *proto.DistributedRelation) *DistributedRelation {
-	return &DistributedRelation{
-		Name:        rel.Name,
-		ColumnNames: rel.Columns,
+	rdistr := &DistributedRelation{
+		Name: rel.Name,
+	}
+
+	for _, e := range rel.DistributionKey {
+		rdistr.DistributionKey = append(rdistr.DistributionKey, DistributionKeyEntry{
+			Column:       e.Column,
+			HashFunction: e.HashFunction,
+		})
 	}
+
+	return rdistr
+}
+
+func DistributedRelationFromSQL(rel *spqrparser.DistributedRelation) *DistributedRelation {
+	rdistr := &DistributedRelation{
+		Name: rel.Name,
+	}
+
+	for _, e := range rel.DistributionKey {
+		rdistr.DistributionKey = append(rdistr.DistributionKey, DistributionKeyEntry{
+			Column:       e.Column,
+			HashFunction: e.HashFunction,
+		})
+	}
+
+	return rdistr
 }
 
 type Distribution struct {
@@ -77,10 +130,7 @@ func DistributionFromProto(ds *proto.Distribution) *Distribution {
 func DistributionToProto(ds *Distribution) *proto.Distribution {
 	drels := make([]*proto.DistributedRelation, 0)
 	for _, r := range ds.Relations {
-		drels = append(drels, &proto.DistributedRelation{
-			Name:    r.Name,
-			Columns: r.ColumnNames,
-		})
+		drels = append(drels, DistributedRelatitonToProto(r))
 	}
 	return &proto.Distribution{
 		Id:          ds.Id,