Merge pull request groue#1700 from groue/dev/all-columns-excluding

Select all columns from a table, but a few ones

Author: groue

Documentation/QueryInterfaceOrganization.md

@@ -497,15 +497,15 @@ protocol SQLSelectable {
 SQLSelectable feeds the `select()` method of the query interface:
 
 ```swift
-Player.select(AllColumns())
+Player.select(.allColumns)
 Player.select(Column("name"), Column("score"))
 ```
 
 All [SQLSpecificExpressible] values are selectable. Other selectable values are:
 
 ```swift
 // SELECT * FROM player
-Player.select(AllColumns())
+Player.select(.allColumns)
 
 // SELECT MAX(score) AS maxScore FROM player
 Player.select(max(Column("score")).forKey("maxScore"))

GRDB/Documentation.docc/DatabaseSchemaRecommendations.md

@@ -151,7 +151,7 @@ extension Player: FetchableRecord, MutablePersistableRecord {
     // Required because the primary key
     // is the hidden rowid column.
     static var databaseSelection: [any SQLSelectable] {
-        [AllColumns(), Column.rowID]
+        [.allColumns, .rowID]
     }
 
     // Update id upon successful insertion

GRDB/QueryInterface/SQL/Column.swift

@@ -61,6 +61,13 @@ extension ColumnExpression {
 
 extension ColumnExpression where Self == Column {
     /// The hidden rowID column.
+    ///
+    /// For example:
+    ///
+    /// ```swift
+    /// // SELECT rowid FROM player
+    /// let rowids = try Player.select(.rowID).fetchSet(db)
+    /// ```
     public static var rowID: Self { Column.rowID }
 }
 

GRDB/QueryInterface/SQL/SQLExpression.swift

@@ -2136,7 +2136,14 @@ public protocol SQLExpressible {
 }
 
 extension SQLExpressible where Self == Column {
-    /// The hidden rowID column
+    /// The hidden rowID column.
+    ///
+    /// For example:
+    ///
+    /// ```swift
+    /// // SELECT rowid FROM player
+    /// let rowids = try Player.select(.rowID).fetchSet(db)
+    /// ```
     public static var rowID: Self { Column.rowID }
 }
 

GRDB/QueryInterface/SQL/SQLRelation.swift

@@ -636,7 +636,7 @@ extension SQLRelation {
             }
 
             // <https://github.com/groue/GRDB.swift/issues/1357>
-            guard selection.allSatisfy(\.isTriviallyCountable) else {
+            if selection.contains(where: \.requiresTrivialCount) {
                 return try fetchTrivialCount(db)
             }