Reflection very slow and using a lot of memory

[grahamburgsma]

I have a table with over 100 columns. Using migrations addProperties(to: builder) has no issues, but when specifying each column using builder.field(for: \.myColumn), then bad things happen.

Memory usage climbs to over 100mb and the execution is frozen for ~1 minute. From profiling, I believe CodableReflection is the cause of this, specifically this function:
public static func anyDecodeProperty(valueType: Any.Type, keyPath: AnyKeyPath) throws -> ReflectedProperty?

[calebkleveter]

The reflection implementation has been rebuilt in CodableKit which will be used in Vapor 4, so hopefully it has already been fixed there.

In the mean time, we should try to track down the issue in Core and patch it.

[grahamburgsma]

The reflection implementation has been rebuilt in CodableKit which will be used in Vapor 4, so hopefully it has already been fixed there.

I did a test using CodableKit and didn't encounter the performance issues as seen with the current implementation in Vapor 3, so that's promising.