bowenli86 commented on issue #9450: [FLINK-13711][sql-client] Hive array values 
not properly displayed in…
   > > The change looks fine. However, I'm wondering if we need to convert 
recursively because otherwise nested non-primitive types will be still showed 
as non-meaningful stuff.
   > @xuefuz `Arrays.deepToString` will take care of that for us as indicated 
by the JavaDoc:
   > ```
   >      * Returns a string representation of the "deep contents" of the 
   >      * array.  If the array contains other arrays as elements, the string
   >      * representation contains their contents and so on.  This method is
   >      * designed for converting multidimensional arrays to strings.
   > ```
   in that case, can we add some unit tests to verify that?

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.
For queries about this service, please contact Infrastructure at:

With regards,
Apache Git Services

Reply via email to