You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/01/13 18:58:22 UTC

[GitHub] [arrow] wjones127 opened a new pull request #12148: [Python] Add character limit to Table.to_string()

wjones127 opened a new pull request #12148:
URL: https://github.com/apache/arrow/pull/12148


   This prevents table column preview from being too long.
   
   The main disadvantage is there is not way to guarantee that we don't truncate the preview in the middle of a value. For example, `[20, 30, 40]` could be truncated to `[20, 30, 4...`, which might be a little misleadning.


-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] edponce commented on a change in pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
edponce commented on a change in pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#discussion_r786948934



##########
File path: python/pyarrow/table.pxi
##########
@@ -1342,10 +1344,11 @@ cdef class Table(_PandasConvertible):
         if preview_cols:
             pieces.append('----')
             for i in range(min(self.num_columns, preview_cols)):
-                pieces.append('{}: {}'.format(
-                    self.field(i).name,
-                    self.column(i).to_string(indent=0, skip_new_lines=True)
-                ))
+                col_string = self.column(i).to_string(
+                    indent=0, skip_new_lines=True)
+                if len(col_string) > cols_char_limit:
+                    col_string = col_string[:(cols_char_limit - 3)] + '...'
+                pieces.append('{}: {}'.format(self.field(i).name, col_string))

Review comment:
       I understand and agree with the current approach. Simply leaving notes as food for thought or future reference.




-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] edponce commented on a change in pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
edponce commented on a change in pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#discussion_r787096905



##########
File path: python/pyarrow/table.pxi
##########
@@ -1342,10 +1344,11 @@ cdef class Table(_PandasConvertible):
         if preview_cols:
             pieces.append('----')
             for i in range(min(self.num_columns, preview_cols)):
-                pieces.append('{}: {}'.format(
-                    self.field(i).name,
-                    self.column(i).to_string(indent=0, skip_new_lines=True)
-                ))
+                col_string = self.column(i).to_string(
+                    indent=0, skip_new_lines=True)
+                if len(col_string) > cols_char_limit:
+                    col_string = col_string[:(cols_char_limit - 3)] + '...'
+                pieces.append('{}: {}'.format(self.field(i).name, col_string))

Review comment:
       Is there a JIRA/PR for implementing similar functionality in C++ Pretty Printer?




-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] github-actions[bot] commented on pull request #12148: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#issuecomment-1012419893


   <!--
     Licensed to the Apache Software Foundation (ASF) under one
     or more contributor license agreements.  See the NOTICE file
     distributed with this work for additional information
     regarding copyright ownership.  The ASF licenses this file
     to you under the Apache License, Version 2.0 (the
     "License"); you may not use this file except in compliance
     with the License.  You may obtain a copy of the License at
   
       http://www.apache.org/licenses/LICENSE-2.0
   
     Unless required by applicable law or agreed to in writing,
     software distributed under the License is distributed on an
     "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
     KIND, either express or implied.  See the License for the
     specific language governing permissions and limitations
     under the License.
   -->
   
   Thanks for opening a pull request!
   
   If this is not a [minor PR](https://github.com/apache/arrow/blob/master/CONTRIBUTING.md#Minor-Fixes). Could you open an issue for this pull request on JIRA? https://issues.apache.org/jira/browse/ARROW
   
   Opening JIRAs ahead of time contributes to the [Openness](http://theapacheway.com/open/#:~:text=Openness%20allows%20new%20users%20the,must%20happen%20in%20the%20open.) of the Apache Arrow project.
   
   Then could you also rename pull request title in the following format?
   
       ARROW-${JIRA_ID}: [${COMPONENT}] ${SUMMARY}
   
   or
   
       MINOR: [${COMPONENT}] ${SUMMARY}
   
   See also:
   
     * [Other pull requests](https://github.com/apache/arrow/pulls/)
     * [Contribution Guidelines - How to contribute patches](https://arrow.apache.org/docs/developers/contributing.html#how-to-contribute-patches)
   


-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] edponce commented on a change in pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
edponce commented on a change in pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#discussion_r787096905



##########
File path: python/pyarrow/table.pxi
##########
@@ -1342,10 +1344,11 @@ cdef class Table(_PandasConvertible):
         if preview_cols:
             pieces.append('----')
             for i in range(min(self.num_columns, preview_cols)):
-                pieces.append('{}: {}'.format(
-                    self.field(i).name,
-                    self.column(i).to_string(indent=0, skip_new_lines=True)
-                ))
+                col_string = self.column(i).to_string(
+                    indent=0, skip_new_lines=True)
+                if len(col_string) > cols_char_limit:
+                    col_string = col_string[:(cols_char_limit - 3)] + '...'
+                pieces.append('{}: {}'.format(self.field(i).name, col_string))

Review comment:
       Is there a PR for implementing similar functionality in C++ Pretty Printer?




-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] jorisvandenbossche commented on a change in pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
jorisvandenbossche commented on a change in pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#discussion_r786872840



##########
File path: python/pyarrow/table.pxi
##########
@@ -1342,10 +1344,11 @@ cdef class Table(_PandasConvertible):
         if preview_cols:
             pieces.append('----')
             for i in range(min(self.num_columns, preview_cols)):
-                pieces.append('{}: {}'.format(
-                    self.field(i).name,
-                    self.column(i).to_string(indent=0, skip_new_lines=True)
-                ))
+                col_string = self.column(i).to_string(
+                    indent=0, skip_new_lines=True)
+                if len(col_string) > cols_char_limit:
+                    col_string = col_string[:(cols_char_limit - 3)] + '...'
+                pieces.append('{}: {}'.format(self.field(i).name, col_string))

Review comment:
       @edponce I fully agree that, ideally, this truncation is "smart" about where to cut off and add `...`. 
   But in general it's also the question to what extent this is worth the extra complexity (depending on how complex it would be of course). Instead of parsing the string, another option could also be to slice the number of elements before converting to string (although for nested data types that won't necessarily work as desired). 
   
   Now, on the short-term (for 0.7.0), I personally find it more important that we at least do _some_ truncation (because currently the repr can be completely useful / annoying by flooding your terminal)




-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] wjones127 commented on a change in pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
wjones127 commented on a change in pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#discussion_r787110311



##########
File path: python/pyarrow/table.pxi
##########
@@ -1342,10 +1344,11 @@ cdef class Table(_PandasConvertible):
         if preview_cols:
             pieces.append('----')
             for i in range(min(self.num_columns, preview_cols)):
-                pieces.append('{}: {}'.format(
-                    self.field(i).name,
-                    self.column(i).to_string(indent=0, skip_new_lines=True)
-                ))
+                col_string = self.column(i).to_string(
+                    indent=0, skip_new_lines=True)
+                if len(col_string) > cols_char_limit:
+                    col_string = col_string[:(cols_char_limit - 3)] + '...'
+                pieces.append('{}: {}'.format(self.field(i).name, col_string))

Review comment:
       Just created one: https://issues.apache.org/jira/browse/ARROW-15363




-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] github-actions[bot] commented on pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#issuecomment-1012420335


   https://issues.apache.org/jira/browse/ARROW-15329


-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] wjones127 commented on a change in pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
wjones127 commented on a change in pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#discussion_r786980933



##########
File path: python/pyarrow/table.pxi
##########
@@ -1327,6 +1327,8 @@ cdef class Table(_PandasConvertible):
             Display Field-level and Schema-level KeyValueMetadata.
         preview_cols : int, default 0
             Display values of the columns for the first N columns.
+        cols_char_limit : int, default 200

Review comment:
       Thanks.




-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] edponce commented on a change in pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
edponce commented on a change in pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#discussion_r784329846



##########
File path: python/pyarrow/table.pxi
##########
@@ -1342,10 +1344,11 @@ cdef class Table(_PandasConvertible):
         if preview_cols:
             pieces.append('----')
             for i in range(min(self.num_columns, preview_cols)):
-                pieces.append('{}: {}'.format(
-                    self.field(i).name,
-                    self.column(i).to_string(indent=0, skip_new_lines=True)
-                ))
+                col_string = self.column(i).to_string(
+                    indent=0, skip_new_lines=True)
+                if len(col_string) > cols_char_limit:
+                    col_string = col_string[:(cols_char_limit - 3)] + '...'
+                pieces.append('{}: {}'.format(self.field(i).name, col_string))

Review comment:
       Ideally, we could always truncate "correctly" and consistently, that is, truncate at a delimiter and show closing brackets: `[[1,2,3,4,...]]`
   
   But several questions first:
   * Is the delimiter always a comma or known in a variable?
   * Is the table representation always encoded as a list of lists? Or can there be a less/more nesting?
   
   Assuming "yes" for the questions above. There are several cases that can occur:
   1. No truncation: `[[1,2,3,4]]`
   2. Truncate does not reaches the final end bracket (# of brackets is arbitrary): `[[1,2,3,4]...`
   3. Truncate mid-value: `[10,20,30,4...`
   3. Truncate at a value delimiter: `[[1,2,3,...`
   4. Truncate at a list delimiter: `[[1,2,3,4],...`
   5. Truncate at a list ending bracket: `[[1,2,3,4]...`
   
   To display all truncated cases "correctly" and consistently, after slicing the `cols_char_limit`, you would need
   * a stack to keep track of open/close bracket pairs
   * check which symbol is truncation occurring on.
   
   Then resolve as follows:
   * If it is a delimiter, then add `...` and match with closing brackets
   * If it is a closing bracket
      * and there are only closing bracket left, then add them to obtain the full representation.
      * add `...` and match with closing brackets
   * Else it is truncating a value, so remove (or add) characters until reaching a delimiter/bracket
   Most of these checks can be identified with regex.




-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] pitrou commented on pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
pitrou commented on pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#issuecomment-1083158680


   @jorisvandenbossche Do you want to take a look at this PR?


-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] edponce commented on a change in pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
edponce commented on a change in pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#discussion_r786952451



##########
File path: python/pyarrow/table.pxi
##########
@@ -1327,6 +1327,8 @@ cdef class Table(_PandasConvertible):
             Display Field-level and Schema-level KeyValueMetadata.
         preview_cols : int, default 0
             Display values of the columns for the first N columns.
+        cols_char_limit : int, default 200

Review comment:
       The default in the docstring does not matches the value in function arguments.




-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] edponce commented on a change in pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
edponce commented on a change in pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#discussion_r784329846



##########
File path: python/pyarrow/table.pxi
##########
@@ -1342,10 +1344,11 @@ cdef class Table(_PandasConvertible):
         if preview_cols:
             pieces.append('----')
             for i in range(min(self.num_columns, preview_cols)):
-                pieces.append('{}: {}'.format(
-                    self.field(i).name,
-                    self.column(i).to_string(indent=0, skip_new_lines=True)
-                ))
+                col_string = self.column(i).to_string(
+                    indent=0, skip_new_lines=True)
+                if len(col_string) > cols_char_limit:
+                    col_string = col_string[:(cols_char_limit - 3)] + '...'
+                pieces.append('{}: {}'.format(self.field(i).name, col_string))

Review comment:
       Ideally, we could always truncate "correctly" and consistently, that is, truncate at a delimiter and show closing brackets: `[[1,2,3,4,...]]`
   
   But several questions first:
   * Is the delimiter always a comma or known in a variable?
   * Is the table representation always encoded as a list of lists? Or can there be a less/more nesting?
   
   Assuming "yes" for the questions above. There are several cases that can occur:
   1. No truncation: `[[1,2,3,4]]`
   2. Truncate does not reaches the final end bracket (# of brackets is arbitrary): `[[1,2,3,4]...`
   3. Truncate mid-value: `[10,20,30,4...`
   3. Truncate at a value delimiter: `[[1,2,3,...`
   4. Truncate at a list delimiter: `[[1,2,3,4],...`
   5. Truncate at a list ending bracket: `[[1,2,3,4]...`
   
   To display all truncated cases "correctly" and consistently, after slicing the `cols_char_limit`, you would need
   * a stack to keep track of open/close bracket pairs
   * check which symbol is truncation occurring on.
   
   Then resolve as follows:
   * If it is a delimiter, then add `...` and match with closing brackets
   * If it is a closing bracket
      * and there are only closing bracket left, then add them to obtain the full representation.
      * add `...` and match with closing brackets
   * Else it is truncating a value, so remove (or add) characters until reaching a delimiter/bracket
   
   Most of these checks can be identified with regex.




-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] edponce commented on a change in pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
edponce commented on a change in pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#discussion_r784329846



##########
File path: python/pyarrow/table.pxi
##########
@@ -1342,10 +1344,11 @@ cdef class Table(_PandasConvertible):
         if preview_cols:
             pieces.append('----')
             for i in range(min(self.num_columns, preview_cols)):
-                pieces.append('{}: {}'.format(
-                    self.field(i).name,
-                    self.column(i).to_string(indent=0, skip_new_lines=True)
-                ))
+                col_string = self.column(i).to_string(
+                    indent=0, skip_new_lines=True)
+                if len(col_string) > cols_char_limit:
+                    col_string = col_string[:(cols_char_limit - 3)] + '...'
+                pieces.append('{}: {}'.format(self.field(i).name, col_string))

Review comment:
       Ideally, we could always truncate "correctly" and consistently, that is, truncate at a delimiter and show closing brackets: `[[1,2,3,4,...]]`
   
   But several questions first:
   * Is the delimiter always a comma or known in a variable?
   * Is the table representation always encoded as a list of lists? Or can there be a less/more nesting?
   
   Assuming "yes" for the questions above. There are several cases that can occur:
   1. No truncation: `[[1,2,3,4]]`
   2. Truncate does not reaches the final end bracket (# of brackets is arbitrary): `[[1,2,3,4]...`
   3. Truncate mid-value: `[10,20,30,4...`
   3. Truncate at a value delimiter: `[[1,2,3,...`
   4. Truncate at a list delimiter: `[[1,2,3,4],...`
   5. Truncate at a list ending bracket: `[[1,2,3,4]...`
   
   To display all truncated cases "correctly" and consistently, after slicing the `cols_char_limit`, you would need
   * a stack to keep track of open/close bracket pairs
   * check which symbol is truncation occurring on.
   
   Then resolve as follows:
   * If it is a delimiter, then add `...` and match with closing brackets
   * If it is a closing bracket
      * and there are only closing bracket left, then add them to obtain the full representation.
      * add `...` and match with closing brackets
   * Else it is truncating a value, so remove (or add) characters until reaching a delimiter/bracket




-- 
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: github-unsubscribe@arrow.apache.org

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



[GitHub] [arrow] wjones127 commented on a change in pull request #12148: ARROW-15329: [Python] Add character limit to Table.to_string()

Posted by GitBox <gi...@apache.org>.
wjones127 commented on a change in pull request #12148:
URL: https://github.com/apache/arrow/pull/12148#discussion_r786980475



##########
File path: python/pyarrow/table.pxi
##########
@@ -1342,10 +1344,11 @@ cdef class Table(_PandasConvertible):
         if preview_cols:
             pieces.append('----')
             for i in range(min(self.num_columns, preview_cols)):
-                pieces.append('{}: {}'.format(
-                    self.field(i).name,
-                    self.column(i).to_string(indent=0, skip_new_lines=True)
-                ))
+                col_string = self.column(i).to_string(
+                    indent=0, skip_new_lines=True)
+                if len(col_string) > cols_char_limit:
+                    col_string = col_string[:(cols_char_limit - 3)] + '...'
+                pieces.append('{}: {}'.format(self.field(i).name, col_string))

Review comment:
       Thanks for the feedback. I implemented a very basic version of this for now. This looks pretty good for this example:
   
   ```python
   >>> from random import sample, choice
   >>> import pyarrow as pa
   >>> arr_int = pa.array(range(50))
   >>> tree_parts = ["roots", "trunk", "crown", "seeds"]
   >>> arr_list = pa.array([sample(tree_parts, k=choice(range(len(tree_parts)))) for _ in range(50)])
   >>> arr_struct = pa.StructArray.from_arrays([arr_int, arr_list], names=['int_nested', 'list_nested'])
   >>> arr_map = pa.array(
   ...     [
   ...         [(part, choice(range(10))) for part in sample(tree_parts, k=choice(range(len(tree_parts))))]
   ...         for _ in range(50)
   ...     ],
   ...     type=pa.map_(pa.utf8(), pa.int64())
   ... )
   >>> table = pa.table({
   ...     'int': pa.chunked_array([arr_int] * 10),
   ...     'list': pa.chunked_array([arr_list] * 10),
   ...     'struct': pa.chunked_array([arr_struct] * 10),
   ...     'map': pa.chunked_array([arr_map] * 10),
   ... })
   >>> print(table)
   pyarrow.Table
   int: int64
   list: list<item: string>
     child 0, item: string
   struct: struct<int_nested: int64, list_nested: list<item: string>>
     child 0, int_nested: int64
     child 1, list_nested: list<item: string>
         child 0, item: string
   map: map<string, int64>
     child 0, entries: struct<key: string not null, value: int64> not null
         child 0, key: string not null
         child 1, value: int64
   ----
   int: [[0,1,2,3,4,5,6,7,8,9,...,40,41,42,43,44,45,46,47,48,49],[0,1,2,3,4,5,6,7,8,9,...,40,41,42,43,44,45,46,47,48,49],[0,1,2,3,...]...]
   list: [[["seeds","trunk","roots"],["trunk","crown"],["crown"],["trunk"],["crown"],[],["roots","seeds"],["roots"],["trunk","roots"]...]...]
   struct: [  -- is_valid: all not null  -- child 0 type: int64
       [
         0,
         1,
         2,
         3,
         4,
         5,
         6,...]...]
   map: [[    keys:["seeds","crown","trunk"]values:[7,8,7],    keys:["roots","crown"]values:[8,4],    keys:["crown","roots","trunk"]...]...]
   ```
   
   
   The unfortunate thing is it will have bad behavior in the case of string columns containing `[`. For example,
   
   ```python
   >>> pa.table({'x': pa.array(["[" * 100]* 500)})
   x: [["[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[","[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[[",...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]...]
 ...]...]...]...]...]...]
   ```
   
   I think that kind of behavior is pretty unavoidable until we push this limit into the PrettyPrinter implementation itself.
   




-- 
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: github-unsubscribe@arrow.apache.org

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