dariocurr opened a new pull request, #18043:
URL: https://github.com/apache/datafusion/pull/18043

   ## Which issue does this PR close?
   
   - Closes #18042 
   
   ## Rationale for this change
   
   This PR introduces a new dialect enum to improve type safety and code 
maintainability when handling different SQL dialects in DataFusion
   
   1. Provide compile-time guarantees for dialect handling
   2. Improve code readability and self-documentation
   3. Enable better IDE support and autocomplete
   
   ## What changes are included in this PR?
   
   - Added a new `Dialect` enum to represent supported SQL dialects
   - Refactored existing code to use the new enum instead of previous 
representations
   - Modified tests to work with the new enum-based approach
   
   ## Are these changes tested?
   
   Yes
   
   ## Are there any user-facing changes?
   
   Yes, the type of the `dialect` field changed from `String` to `Dialect`


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to