You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Change return value of Table#save when saving to csv files.
Current code returns @table.raw_records (Array of row Arrays). When saving a very big Table, it takes long time to display a large Array with interactive execution such as irb or Jupyter.
When we save as 'arrow' or 'arrows', it simply returns 'true'.
Candidates of return value:
true
self
size of Table
Impact of this request
It will improve the user experience of interactive execution. The return value of #save is rarely used.
# Rationale for this change
Change return value of Table#save when saving Table to csv files.
# What changes are included in this PR?
Change return value to self (`@ table`).
# Are these changes tested?
Add test for saving to csv.
# Are there any user-facing changes?
Return value of Table#save is undetermined.
* Closes: #15289
Authored-by: Hirokazu SUZUKI <heronshoes877@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Describe the enhancement requested
Target method
Arrow::Table#save
Proposed feature
Change return value of Table#save when saving to csv files.
Current code returns
@table.raw_records
(Array of row Arrays). When saving a very big Table, it takes long time to display a large Array with interactive execution such as irb or Jupyter.When we save as 'arrow' or 'arrows', it simply returns 'true'.
Candidates of return value:
Impact of this request
It will improve the user experience of interactive execution. The return value of #save is rarely used.
Reference of this request
https://github.com/apache/arrow/blob/master/ruby/red-arrow/lib/arrow/table-saver.rb#L174-L176
Component(s)
Ruby
The text was updated successfully, but these errors were encountered: