[GitHub] [shardingsphere] RaigorJiang commented on pull request #20843: Add convert shadow distsql and test case

[GitBox]

RaigorJiang commented on PR #20843:
URL: https://github.com/apache/shardingsphere/pull/20843#issuecomment-1240128609

   > About CI error, this is so weird. I have adjusted the order of the 
parameters in the shadow tables properties. The picture is the result of the 
printout of my code.
   > 
   > <img alt="Screen Shot 2022-09-08 at 09 04 10" width="1511" 
src="https://user-images.githubusercontent.com/57054352/189011010-95d0b5ca-9c65-4432-a90a-682691dad954.png";>
   > 
   > <img alt="Screen Shot 2022-09-08 at 09 23 05" width="1511" 
src="https://user-images.githubusercontent.com/57054352/189012927-8c2f70d0-cddf-4dc1-b63d-b6c55bc175c7.png";>
   
   Hi @isHuangXin 
   Because Properties is an inherited HashTable, keys are not guaranteed in 
order.
   Can we compare line by line here, and then use contains to make an assertion 
on the Properties line


-- 
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: notifications-unsubscr...@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org