findepi commented on code in PR #13197:
URL: https://github.com/apache/datafusion/pull/13197#discussion_r1824095533


##########
datafusion/sqllogictest/test_files/string/string_query.slt.part:
##########
@@ -903,20 +991,25 @@ xxxxxxxxxxxXiangpeng NULL ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅdatafusionๆ•ฐๆฎ่žๅˆ NULL
 xxxxxxxxxxxxxRaphael NULL datafusionะ”ะฐั‚ะฐะคัƒัะธะพะฝ NULL
 xxxxxxxxxunder_score NULL ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅun iล›ฤ‡ core NULL
 xxxxxxxxxxxxxpercent NULL pan Tadeusz ma iล›ฤ‡ w NULL
+xxxxxxxxxxxxxxxxxxxx NULL ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ๐Ÿ”ฅ NULL
+NULL NULL NULL NULL
 NULL NULL NULL NULL
 
-query TT
+query TTT
 SELECT
   LPAD(ascii_1, 20),
-  LPAD(unicode_1, 20)
+  LPAD(unicode_1, 20),
+  '|'

Review Comment:
   Added this to diagnose failure 
(https://github.com/apache/datafusion/pull/13197#issuecomment-2449343727), but 
the failure was without this too.
   it's better to keep this though. Editors like removing empty trailing 
whitespace.



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