[ http://opensource.atlassian.com/projects/hibernate/browse/ANN-122?page=co... ] Nicolás Lichtmaier commented on ANN-122: ---------------------------------------- The unrelated change you mention was the only one in the patch. It was done automatically by my Eclipse and it seemed ok, but I'll remove it. Why is it important to handle ordering in natural ids? To "optimize" the index putting some fields first? I don't see that posibility in @Index... In which way it wouldn't be predictable? Some doubts I have with the patch is the way I pass the "mutable" attribute. It looks a generic argument, but I'm only passing it from the natural id handling. That's because I have declared as "Boolean" so that it can be null (which means... I don't know here if this should be mutable). In any case? What would you say I should modify in order to make this suitable for inclusion? -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/hibernate/secure/Administrators.... - For more information on JIRA, see: http://www.atlassian.com/software/jira