paleolimbot commented on code in PR #13541:
URL: https://github.com/apache/arrow/pull/13541#discussion_r923857332
##########
r/R/query-engine.R:
##########
@@ -259,9 +260,40 @@ ExecPlan <- R6Class("ExecPlan",
...
)
},
+ ToString = function() ExecPlan_ToString(self),
Review Comment:
I see your point, although in this case I think the 5 extra lines of code
could be added in when that functionality is implemented (right now they are
just a little confusing). Again, you could choose to make the value accessible,
test it, and document why it needs to exist (I think you should just take it
out though).
--
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-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
