here sort_column is declared as STRING ... and your assign a STRING_32
so there will be an implicit conversion from STRING_32 to STRING_8 ... which may be bad if you have a unicode text.

For now, this is not critical, but whenever you may have Unicode, you should use STRING_32
so this would require many changes in your current code, for now this is not critical, but try to keep that in mind for later.