Allow parsing any permutation of create sequence options

[PartiallyUntyped]

Hi, many thanks for the project!

I would like to be able to parse the create sequence options in any particular order instead of just what postgreslq/duckdb support. Afaik oracle for example allows any permutation of the options.

If this is something that you'd be willing accept, I would be happy to implement the necessary changes to allow any permutation and hide it behind something like a dialect flag.

[alamb]

I would like to be able to parse the create sequence options in any particular order instead of just what postgreslq/duckdb support. Afaik oracle for example allows any permutation of the options.

If this is something that you'd be willing accept, I would be happy to implement the necessary changes to allow any permutation and hide it behind something like a dialect flag.

Sounds like a good feature to me -- thank you 🙏

Given the description of the issue I don't think we need to hide this behind a dialect flag FWIW, though you could do so if you prefer

[PartiallyUntyped]

My concern is that it would create false positives, and potentially allow bad code compilation, eg parsing bad postgresql code and then compiling that back in the wrong order.

I am thinking that this should be controlled on a per dialect basis, and each dialect can control the ordering of acceptable fields.

[alamb]

My concern is that it would create false positives, and potentially allow bad code compilation, eg parsing bad postgresql code and then compiling that back in the wrong order.

That is a reasonable concern, though I think there are examples in the crate where that already happens

I don't have a strong preference