wip

FerretDB · Jul 6, 2022 · 0a29aed · 0a29aed
1 parent 426b00d
commit 0a29aed
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 8 deletions.
diff --git a/README.md b/README.md
@@ -37,7 +37,8 @@ and [contributing guidelines](CONTRIBUTING.md).
 
 * collection name cannot start with the reserved prefix `_ferretdb_`.
 * collection name must not include non-latin letters, spaces, dots, dollars or dashes.
-* collection name length must be less than 119 symbols.
+* collection name length must be less or equal than 119 symbols.
+* collection name must not start with a number.
 
 If you encounter some other difference in behavior, please [join our community](#community) to report a problem.
 

diff --git a/internal/handlers/pg/pgdb/pool.go b/internal/handlers/pg/pgdb/pool.go
@@ -278,7 +278,7 @@ func (pgPool *Pool) Tables(ctx context.Context, schema string) ([]string, error)
 
 	filtered := make([]string, 0, len(tables))
 	for _, table := range tables {
-		if strings.HasPrefix(table, collectionPrefix) {
+		if strings.HasPrefix(table, reservedCollectionPrefix) {
 			continue
 		}
 
@@ -353,15 +353,15 @@ func (pgPool *Pool) DropDatabase(ctx context.Context, db string) error {
 // CreateCollection creates a new FerretDB collection in existing schema.
 //
 // It returns:
-//  - ErrInvalidTableName - if the table name doesn't conform to restrictions.
-//  - ErrAlreadyExist - if table already exist.
-//  - ErrTableNotExist - is schema does not exist.
+//  * ErrInvalidTableName - if the table name doesn't conform to restrictions.
+//  * ErrAlreadyExist - if table already exist.
+//  * ErrTableNotExist - is schema does not exist.
 func (pgPool *Pool) CreateCollection(ctx context.Context, db, collection string) error {
 	if !validateCollectionNameRe.MatchString(collection) {
 		return ErrInvalidTableName
 	}
 
-	if strings.HasPrefix(collection, collectionPrefix) {
+	if strings.HasPrefix(collection, reservedCollectionPrefix) {
 		return ErrInvalidTableName
 	}
 

diff --git a/internal/handlers/pg/pgdb/settings.go b/internal/handlers/pg/pgdb/settings.go
@@ -32,10 +32,10 @@ import (
 
 const (
 	// Internal collections prefix.
-	collectionPrefix = "_ferretdb_"
+	reservedCollectionPrefix = "_ferretdb_"
 
 	// Settings table name.
-	settingsTableName = collectionPrefix + "settings"
+	settingsTableName = reservedCollectionPrefix + "settings"
 
 	// PostgreSQL max table name length.
 	maxTableNameLength = 63