Github user svanteschubert commented on the issue:
https://github.com/apache/tinkerpop/pull/892
Long sounds fine. Good catch!
The sort algorithm I suggested is indeed a heuristic, still a better
heuristic as the default String sort by Java ;-)
Ordering ["a100","100a","1a1"] in my head seems not easy and is more a
guessing, I am happy with the given result, yes! ;-)
Okay, I agree that performance penalty by default is a suboptimal idea,
then let's go the other way and make debug/normalization easier accessible by
command line. I suggest adding the debug option even in the [TinkerPop Docu
GraphML initial
mentioning](tinkerpop.apache.org/docs/current/reference/#_graphml_reader_writer)
(I suggest to add this option for other formats as JSON as well).
The new normalized output is more predictable (at least for me) ;-) and
would ease the analysis of regressions tests as I plan to do for my usage.
Indeed, not mandatory but I find it annoying enough to spend my weekend fixing
it...
PS: Sorry for the import reordering, using Netbeans and the reording of
imports happens on save, I will install the latest Apache Netbeans version and
see how I prevent this noice from happening next time!
Thanks for your quick review, Stephen!
Svante
---
