MSSQL: Clarify Synapse final-column trailing-comma support

Ouverte
  • Quelque chose ne va pas ?
#2,605 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
35/100
Type d'issue
Fonctionnalité
Clarté
À clarifier
Activité
Active
Stack technique
azure, rust, sql
Domaine
databases, devtools

Piste de recherche

Check the referenced Azure Synapse CREATE TABLE documentation and the MsSqlDialect handling of CREATE TABLE syntax first; compare the requested final-column case with related generic trailing-comma support in #1212. Done means the scope is confirmed for Synapse serverless temporary tables, while existing ParserOptions behavior and rejection of malformed or unrelated trailing arguments remain preserved.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

SQL Server

MSSQL: Clarify Synapse final-column trailing-comma support

Should MsSqlDialect have a narrowly scoped capability for a final comma after a complete CREATE TABLE column definition?

Synthetic reproducer:

CREATE TABLE #example (
    id INT,
    label VARCHAR(32),
);

The default MSSQL parser rejects this. The proposed scope is Azure Synapse serverless SQL pool temporary tables, not every SQL Server variant.
I have not independently executed this reproducer against that service. The documented CREATE TABLE grammar does not clearly establish this allowance, so I am requesting scope confirmation before proposing a default dialect change.

Reference: https://learn.microsoft.com/en-us/sql/t-sql/statements/create-table-transact-sql?view=azure-sqldw-latest

If supported, the capability should apply only after a complete final column definition, without enabling global trailing commas, malformed/doubled commas, or trailing function arguments. Existing explicit ParserOptions behavior should remain unchanged.
Related generic trailing-comma support exists in #1212; this question concerns a narrow default capability and engine scope.

Prepared with GitHub Copilot assistance. All material above is synthetic.

Langage dominant
Rust
Étoiles
3.5k
Forks
805
Merge moyen
2 j 3 h
PR mergées (30 j)
116

Préparer son environnement

  • Aucun Dockerfile ni fichier Docker Compose
  • Propose un modèle de pull request
  • Aucun guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de apache/datafusion-sqlparser-rs

Toutes les issues de apache/datafusion-sqlparser-rs

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.