>The only problem is that if column is nullable, we need to also check if NULLability has changed as well (as that code does with extra 2 conditions)

In practice, columns that are candidates for logging tend to have constraints against NULL values (or in most cases, they should), but that is a good point, thanks.