after some discussion with the team, we think it would better to not having this change in 4.1/master branch, the better place for such change should be 5.0, aka, metamodel branch.

paste some logs here:

23:07 (sebersole) if you want to ad that for 5.0, where those chanes are is not the right place
23:07 (sebersole) i can point you to the right place if you need
23:08 emmanuel has left IRC (Read error: Operation timed out)
23:08 (stliu) sebersole: i will work on that revert thing
23:08 (stliu) org.hibernate.tool.schema.internal.StandardTableExporter?
23:09 (sebersole) stliu: well thats one option
23:09 (sebersole) but
23:09 (sebersole) i think i'd rather the columns be sorted in memory
23:09 (sebersole) so everything sees the same order
23:10 (sebersole) that would not affect PKs
23:10 (sebersole) and like i said, i really do not get the point wrt FKs
23:12 (sebersole) not sure if it is still true, but it used to be better (perf wise) to read columns in the same order from the table def
23:12 (sebersole) on some dbs