Agree with the points Felix made. One thing is that it looks like the only problem is vignettes and the tests are being skipped as designed. If you see https://win-builder.r-project.org/incoming_pretest/SparkR_2.4.0_20181105_165757/Windows/00check.log and https://win-builder.r-project.org/incoming_pretest/SparkR_2.4.0_20181105_165757/Debian/00check.log, the tests run in 1s. On Tue, Nov 6, 2018 at 1:29 PM Felix Cheung <felixcheun...@hotmail.com> wrote: > > I’d rather not mess with 2.4.0 at this point. On CRAN is nice but users can > also install from Apache Mirror. > > Also I had attempted and failed to get vignettes not to build, it was non > trivial and could t get it to work. It I have an idea. > > As for tests I don’t know exact why is it not skipped. Need to investigate > but worse case test_package can run with 0 test. > > > > ________________________________ > From: Sean Owen <sro...@gmail.com> > Sent: Tuesday, November 6, 2018 10:51 AM > To: Shivaram Venkataraman > Cc: Felix Cheung; Wenchen Fan; Matei Zaharia; dev > Subject: Re: [CRAN-pretest-archived] CRAN submission SparkR 2.4.0 > > I think the second option, to skip the tests, is best right now, if > the alternative is to have no SparkR release at all! > Can we monkey-patch the 2.4.0 release for SparkR in this way, bless it > from the PMC, and release that? It's drastic but so is not being able > to release, I think. > Right? or is CRAN not actually an important distribution path for > SparkR in particular? > > On Tue, Nov 6, 2018 at 12:49 PM Shivaram Venkataraman > <shiva...@eecs.berkeley.edu> wrote: > > > > Right - I think we should move on with 2.4.0. > > > > In terms of what can be done to avoid this error there are two strategies > > - Felix had this other thread about JDK 11 that should at least let > > Spark run on the CRAN instance. In general this strategy isn't > > foolproof because the JDK version and other dependencies on that > > machine keep changing over time and we dont have much control over it. > > Worse we also dont have much control > > - The other solution is to not run code to build the vignettes > > document and just have static code blocks there that have been > > pre-evaluated / pre-populated. We can open a JIRA to discuss the > > pros/cons of this ? > > > > Thanks > > Shivaram > > > > On Tue, Nov 6, 2018 at 10:57 AM Felix Cheung <felixcheun...@hotmail.com> > > wrote: > > > > > > We have not been able to publish to CRAN for quite some time (since 2.3.0 > > > was archived - the cause is Java 11) > > > > > > I think it’s ok to announce the release of 2.4.0 > > > > > > > > > ________________________________ > > > From: Wenchen Fan <cloud0...@gmail.com> > > > Sent: Tuesday, November 6, 2018 8:51 AM > > > To: Felix Cheung > > > Cc: Matei Zaharia; Sean Owen; Spark dev list; Shivaram Venkataraman > > > Subject: Re: [CRAN-pretest-archived] CRAN submission SparkR 2.4.0 > > > > > > Do you mean we should have a 2.4.0 release without CRAN and then do a > > > 2.4.1 immediately? > > > > > > On Wed, Nov 7, 2018 at 12:34 AM Felix Cheung <felixcheun...@hotmail.com> > > > wrote: > > >> > > >> Shivaram and I were discussing. > > >> Actually we worked with them before. Another possible approach is to > > >> remove the vignettes eval and all test from the source package... in the > > >> next release. > > >> > > >> > > >> ________________________________ > > >> From: Matei Zaharia <matei.zaha...@gmail.com> > > >> Sent: Tuesday, November 6, 2018 12:07 AM > > >> To: Felix Cheung > > >> Cc: Sean Owen; dev; Shivaram Venkataraman > > >> Subject: Re: [CRAN-pretest-archived] CRAN submission SparkR 2.4.0 > > >> > > >> Maybe it’s wroth contacting the CRAN maintainers to ask for help? > > >> Perhaps we aren’t disabling it correctly, or perhaps they can ignore > > >> this specific failure. +Shivaram who might have some ideas. > > >> > > >> Matei > > >> > > >> > On Nov 5, 2018, at 9:09 PM, Felix Cheung <felixcheun...@hotmail.com> > > >> > wrote: > > >> > > > >> > I don¡Št know what the cause is yet. > > >> > > > >> > The test should be skipped because of this check > > >> > https://github.com/apache/spark/blob/branch-2.4/R/pkg/inst/tests/testthat/test_basic.R#L21 > > >> > > > >> > And this > > >> > https://github.com/apache/spark/blob/branch-2.4/R/pkg/inst/tests/testthat/test_basic.R#L57 > > >> > > > >> > But it ran: > > >> > callJStatic("org.apache.spark.ml.r.GeneralizedLinearRegressionWrapper", > > >> > "fit", formula, > > >> > > > >> > The earlier release was achived because of Java 11+ too so this > > >> > unfortunately isn¡Št new. > > >> > > > >> > > > >> > From: Sean Owen <sro...@gmail.com> > > >> > Sent: Monday, November 5, 2018 7:22 PM > > >> > To: Felix Cheung > > >> > Cc: dev > > >> > Subject: Re: [CRAN-pretest-archived] CRAN submission SparkR 2.4.0 > > >> > > > >> > What can we do to get the release through? is there any way to > > >> > circumvent these tests or otherwise hack it? or does it need a > > >> > maintenance release? > > >> > On Mon, Nov 5, 2018 at 8:53 PM Felix Cheung > > >> > <felixcheun...@hotmail.com> wrote: > > >> > > > > >> > > FYI. SparkR submission failed. It seems to detect Java 11 correctly > > >> > > with vignettes but not skipping tests as would be expected. > > >> > > > > >> > > Error: processing vignette ¡¥sparkr-vignettes.Rmd¡Š failed with > > >> > > diagnostics: > > >> > > Java version 8 is required for this package; found version: 11.0.1 > > >> > > Execution halted > > >> > > > > >> > > * checking PDF version of manual ... OK > > >> > > * DONE > > >> > > Status: 1 WARNING, 1 NOTE > > >> > > > > >> > > Current CRAN status: ERROR: 1, OK: 1 > > >> > > See: > > >> > > <https://CRAN.R-project.org/web/checks/check_results_SparkR.html> > > >> > > > > >> > > Version: 2.3.0 > > >> > > Check: tests, Result: ERROR > > >> > > Running ¡¥run-all.R¡Š [8s/35s] > > >> > > Running the tests in ¡¥tests/run-all.R¡Š failed. > > >> > > Last 13 lines of output: > > >> > > 4: > > >> > > callJStatic("org.apache.spark.ml.r.GeneralizedLinearRegressionWrapper", > > >> > > "fit", formula, > > >> > > data@sdf, tolower(family$family), family$link, tol, > > >> > > as.integer(maxIter), weightCol, > > >> > > regParam, as.double(var.power), as.double(link.power), > > >> > > stringIndexerOrderType, > > >> > > offsetCol) > > >> > > 5: invokeJava(isStatic = TRUE, className, methodName, ...) > > >> > > 6: handleErrors(returnStatus, conn) > > >> > > 7: stop(readString(conn)) > > >> > > > > >> > > ùùùù testthat results > > >> > > ùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùùù > > >> > > OK: 0 SKIPPED: 0 FAILED: 2 > > >> > > 1. Error: create DataFrame from list or data.frame (@test_basic.R#26) > > >> > > 2. Error: spark.glm and predict (@test_basic.R#58) > > >> > > > > >> > > > > >> > > > > >> > > ---------- Forwarded message --------- > > >> > > Date: Mon, Nov 5, 2018, 10:12 > > >> > > Subject: [CRAN-pretest-archived] CRAN submission SparkR 2.4.0 > > >> > > > > >> > > Dear maintainer, > > >> > > > > >> > > package SparkR_2.4.0.tar.gz does not pass the incoming checks > > >> > > automatically, please see the following pre-tests: > > >> > > Windows: > > >> > > <https://win-builder.r-project.org/incoming_pretest/SparkR_2.4.0_20181105_165757/Windows/00check.log> > > >> > > Status: 1 NOTE > > >> > > Debian: > > >> > > <https://win-builder.r-project.org/incoming_pretest/SparkR_2.4.0_20181105_165757/Debian/00check.log> > > >> > > Status: 1 WARNING, 1 NOTE > > >> > > > > >> > > Last released version's CRAN status: ERROR: 1, OK: 1 > > >> > > See: > > >> > > <https://CRAN.R-project.org/web/checks/check_results_SparkR.html> > > >> > > > > >> > > CRAN Web: <https://cran.r-project.org/package=SparkR> > > >> > > > > >> > > Please fix all problems and resubmit a fixed version via the webform. > > >> > > If you are not sure how to fix the problems shown, please ask for > > >> > > help on the R-package-devel mailing list: > > >> > > <https://stat.ethz.ch/mailman/listinfo/r-package-devel> > > >> > > If you are fairly certain the rejection is a false positive, please > > >> > > reply-all to this message and explain. > > >> > > > > >> > > More details are given in the directory: > > >> > > <https://win-builder.r-project.org/incoming_pretest/SparkR_2.4.0_20181105_165757/> > > >> > > The files will be removed after roughly 7 days. > > >> > > > > >> > > No strong reverse dependencies to be checked. > > >> > > > > >> > > Best regards, > > >> > > CRAN teams' auto-check service > > >> > > Flavor: r-devel-linux-x86_64-debian-gcc, r-devel-windows-ix86+x86_64 > > >> > > Check: CRAN incoming feasibility, Result: NOTE > > >> > > Maintainer: 'Shivaram Venkataraman <shiva...@cs.berkeley.edu>' > > >> > > > > >> > > New submission > > >> > > > > >> > > Package was archived on CRAN > > >> > > > > >> > > Possibly mis-spelled words in DESCRIPTION: > > >> > > Frontend (4:10, 5:28) > > >> > > > > >> > > CRAN repository db overrides: > > >> > > X-CRAN-Comment: Archived on 2018-05-01 as check problems were not > > >> > > corrected despite reminders. > > >> > > > > >> > > Flavor: r-devel-linux-x86_64-debian-gcc > > >> > > Check: re-building of vignette outputs, Result: WARNING > > >> > > Error in re-building vignettes: > > >> > > ... > > >> > > > > >> > > Attaching package: 'SparkR' > > >> > > > > >> > > The following objects are masked from 'package:stats': > > >> > > > > >> > > cov, filter, lag, na.omit, predict, sd, var, window > > >> > > > > >> > > The following objects are masked from 'package:base': > > >> > > > > >> > > as.data.frame, colnames, colnames<-, drop, endsWith, > > >> > > intersect, rank, rbind, sample, startsWith, subset, summary, > > >> > > transform, union > > >> > > > > >> > > trying URL > > >> > > 'http://mirror.klaus-uwe.me/apache/spark/spark-2.4.0/spark-2.4.0-bin-hadoop2.7.tgz' > > >> > > Content type 'application/octet-stream' length 227893062 bytes > > >> > > (217.3 MB) > > >> > > ================================================== > > >> > > downloaded 217.3 MB > > >> > > > > >> > > Quitting from lines 65-67 (sparkr-vignettes.Rmd) > > >> > > Error: processing vignette 'sparkr-vignettes.Rmd' failed with > > >> > > diagnostics: > > >> > > Java version 8 is required for this package; found version: 11.0.1 > > >> > > Execution halted > > >>
--------------------------------------------------------------------- To unsubscribe e-mail: dev-unsubscr...@spark.apache.org
