This refactors MongoResultSet to use BsonTypeInfo and significantly cleans up the switch statements.
I've also removed the "end_of_document" check from MongoResultSet as it wasn't used in any meaningful way other than it couldn't be converted in any of the switches. Since it was a BsonValue that is being passed into the various conversion methods, there's actually no way for it to be an "end_of_document" since it can't be created as a concrete type to derive a BsonType from (let alone a BsonTypeInfo).
This also updates the go version used in the build task, addressing SQL-1083.
terakilobyte
commented
1 year ago
This refactors
MongoResultSetto useBsonTypeInfoand significantly cleans up the switch statements.I've also removed the "end_of_document" check from
MongoResultSetas it wasn't used in any meaningful way other than it couldn't be converted in any of the switches. Since it was a BsonValue that is being passed into the various conversion methods, there's actually no way for it to be an "end_of_document" since it can't be created as a concrete type to derive aBsonTypefrom (let alone aBsonTypeInfo).This also updates the go version used in the build task, addressing SQL-1083.