jorisvandenbossche commented on a change in pull request #11479:
URL: https://github.com/apache/arrow/pull/11479#discussion_r737306330
##########
File path: docs/source/python/getstarted.rst
##########
@@ -134,6 +141,14 @@ and will lazily load chunks of data only when iterating
over them
for table_chunk in birthdays_dataset.to_batches():
print("AGES", pc.subtract(current_year, table_chunk["years"]))
+.. ipython:: python
+ :suppress:
+
+ import shutil
+
+ shutil.rmtree("savedir")
Review comment:
For this one I could maybe indeed move it before the write call.
Alternatively, I could also use the same approach of using a temporary
directory instead. I think the main disadvantage of that is that it adds quite
some "clutter" to the example (and for that reason I would like to keep it
simple in this getting started page).
Another alternative would be to change the working directory in a hidden
ipython code block just before, and restore it afterwards. That might actually
be the cleanest solution, while keeping the visible part in the docs simple.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
