Re: CRAN package SparkR

2021-09-01 Thread Hyukjin Kwon
Made a quick fix: https://github.com/apache/spark/pull/33887 I would very appreciate if you guys double check and test against my change for doubly sure .. adding @Shivaram Venkataraman too FYI 2021년 9월 1일 (수) 오전 11:56, Felix Cheung 님이 작성: > I think a few lines to add the prompt might be

Re: CRAN package SparkR

2021-08-31 Thread Felix Cheung
I think a few lines to add the prompt might be enough. This checks for interactive() https://github.com/apache/spark/blob/c6a2021fec5bab9069fbfba33f75d4415ea76e99/R/pkg/R/sparkR.R#L658 On Tue, Aug 31, 2021 at 5:55 PM Hyukjin Kwon wrote: > Oh I missed this. Yes, can we simply get the user'

Re: CRAN package SparkR

2021-08-31 Thread Hyukjin Kwon
Oh I missed this. Yes, can we simply get the user' confirmation when we install.spark? IIRC, the auto installation is only triggered by interactive shell so getting user's confirmation should be fine. 2021년 6월 18일 (금) 오전 2:54, Felix Cheung 님이 작성: > Any suggestion or comment on this? They are

Re: CRAN package SparkR

2021-06-17 Thread Felix Cheung
Any suggestion or comment on this? They are going to remove the package by 6-28 Seems to me if we have a switch to opt in to install (and not by default on), or prompt the user in interactive session, should be good as user confirmation. On Sun, Jun 13, 2021 at 11:25 PM Felix Cheung wrote: >