Not sure I understand this change. If you have an UPSERT statement with no columns specified, it means that you're values correspond to the columns in the order they're declared in the schema. I don't think we'd want to enter this if block if dynamic (or static) columns are included, no?