PostgresNotificationSequence is not Sendable in 5.6 (vapor#392)

fabianfett · web-flow · commit 9bddf75192d0 · 2023-08-09T21:11:53.000Z
`AsyncThrowingStream` is not `Sendable` in Swift 5.6. Because of this `PostgresNotificationSequence` can not be `Sendable` in 5.6.
diff --git a/Sources/PostgresNIO/New/PostgresNotificationSequence.swift b/Sources/PostgresNIO/New/PostgresNotificationSequence.swift
@@ -3,7 +3,7 @@ public struct PostgresNotification: Sendable {
     public let payload: String
 }
 
-public struct PostgresNotificationSequence: AsyncSequence, Sendable {
+public struct PostgresNotificationSequence: AsyncSequence {
     public typealias Element = PostgresNotification
 
     let base: AsyncThrowingStream<PostgresNotification, Error>
@@ -20,3 +20,8 @@ public struct PostgresNotificationSequence: AsyncSequence, Sendable {
         }
     }
 }
+
+#if swift(>=5.7)
+// AsyncThrowingStream is marked as Sendable in Swift 5.6
+extension PostgresNotificationSequence: Sendable {}
+#endif

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@ public struct PostgresNotification: Sendable {`
`3`	`3`	`public let payload: String`
`4`	`4`	`}`
`5`	`5`
`6`		`-public struct PostgresNotificationSequence: AsyncSequence, Sendable {`
	`6`	`+public struct PostgresNotificationSequence: AsyncSequence {`
`7`	`7`	`public typealias Element = PostgresNotification`
`8`	`8`
`9`	`9`	`let base: AsyncThrowingStream<PostgresNotification, Error>`
`@@ -20,3 +20,8 @@ public struct PostgresNotificationSequence: AsyncSequence, Sendable {`
`20`	`20`	`}`
`21`	`21`	`}`
`22`	`22`	`}`
	`23`	`+`
	`24`	`+#if swift(>=5.7)`
	`25`	`+// AsyncThrowingStream is marked as Sendable in Swift 5.6`
	`26`	`+extension PostgresNotificationSequence: Sendable {}`
	`27`	`+#endif`