Don't sort variables in TXTResultSerializer

[aucampia]

Fixes #1249

Proposed Changes

• Eliminate sorting of columns in TXTResultSerializer

[coveralls]

Coverage increased (+0.2%) to 75.704% when pulling a14777f on iafork:fix_txtresult_column_order into 8d592a2 on RDFLib:master.

[white-gecko]

Does this introduce any speedup or something like that?

[aucampia]

Does this introduce any speedup or something like that?

I did not test it, I presume it would, but I don't think it will be significant. This sort happens once per query, not per row.

There is not really a production or high volume use case for txtresults as it is not standardized and using csv, tsv, json, xml would be better for such cases.

It is more an issue of correctness, as when I run a select and ouput it as txt it would be best if the column order is the same as the order I specified in my select (same as it would be for csv and tsv as far as I know).

This was mainly prompted by #1249 - I do believe I have been annoyed by this in the past because I recall instead using csv and piping to column.