ankusharora09 commented on issue #17498:
URL: https://github.com/apache/datafusion/issues/17498#issuecomment-3765614902

   I tried this with SET datafusion.explain.format and wanted to confirm 
expected behavior.
   
   I initially ran:
   
   SET datafusion.explain.format = indnet;
   
   
   which appears to be a typo, and then:
   
   SET datafusion.explain.format = indent;
   
   
   Both commands return 0 row(s) fetched, which I understand is expected since 
SET statements only modify session state and don’t return results.
   
   The configuration does seem to take effect when running an EXPLAIN query 
afterward, but the lack of feedback (especially for invalid values like the 
typo) can be a bit confusing from a user perspective.
   
   One possible improvement could be to validate the value passed to 
datafusion.explain.format and either:
   
   warn or error on invalid values, or
   
   provide a short confirmation message indicating the active format.
   
   If this aligns with the project’s direction, I’d be happy to work on adding 
validation or clearer feedback around this setting. Please let me know if this 
would be welcome or if there’s a preferred approach.
   


-- 
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