Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix flytable maximum cells error by adjusting chunksize #147

Merged
merged 1 commit into from
Feb 16, 2022

Conversation

jefferis
Copy link
Collaborator

@jefferis jefferis commented Feb 15, 2022

  • Error in flytable_list_rows("info")
 #> Error in py_call_impl(x, dots$args, dots$keywords): ConnectionError: [Errno 413] {"error_msg":"the number of cells accessing the table exceeds the limit"}
  • turns out that there is now a limit in the number of cells that you can return of 1,000,000
  • reported by @YijieYin
  • @schlegelp FYI

* #> Error in py_call_impl(x, dots$args, dots$keywords): ConnectionError: [Errno 413] {"error_msg":"the number of cells accessing the table exceeds the limit"}
* turns out that there is now a limit in the number of cells that you can return of 1,000,000
@codecov
Copy link

codecov bot commented Feb 15, 2022

Codecov Report

Merging #147 (91707ff) into master (6f10ced) will increase coverage by 0.14%.
The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #147      +/-   ##
==========================================
+ Coverage   47.69%   47.83%   +0.14%     
==========================================
  Files          29       29              
  Lines        3537     3541       +4     
==========================================
+ Hits         1687     1694       +7     
+ Misses       1850     1847       -3     
Impacted Files Coverage Δ
R/flytable.R 67.40% <85.71%> (+1.20%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6f10ced...91707ff. Read the comment docs.

@jefferis jefferis merged commit 70d3b45 into master Feb 16, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant