sha174n commented on PR #44496:
URL: https://github.com/apache/superset/pull/44496#issuecomment-5768848398

   Addressed in 13b46e2 .
   
   @aminghadersohi on the residual: both messages are pinned now. The executor 
parametrize asserts the exact rendered string per case (including a two-head 
case), and the sql_lab test asserts the full message instead of just the 
presence of a head. Reverting either formatting fix kills 10 tests; before this 
it killed 0. Also folded the structured-node head through the same allow-list 
as the command-fallback head, so the two tracks can no longer drift.
   
   @rusackas added the UPDATING.md entry. While writing it I corrected the 
rationale: PUT/GET do file I/O on the query host, but REMOVE/RM delete files 
within the stage, so the shared reason is that none of them read or write table 
data. Behavior is unchanged.


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