Uploaded image for project: 'Spark'
  1. Spark
  2. SPARK-19460

Update dataset used in R documentation, examples to reduce warning noise and confusions

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 2.1.0
    • 2.2.0
    • SparkR
    • None

    Description

      Running build we have a bunch of warnings from using the `iris` dataset, for example.

      Warning in FUN(X[[1L]], ...) :
      Use Sepal_Length instead of Sepal.Length as column name
      Warning in FUN(X[[2L]], ...) :
      Use Sepal_Width instead of Sepal.Width as column name
      Warning in FUN(X[[3L]], ...) :
      Use Petal_Length instead of Petal.Length as column name
      Warning in FUN(X[[4L]], ...) :
      Use Petal_Width instead of Petal.Width as column name
      Warning in FUN(X[[1L]], ...) :
      Use Sepal_Length instead of Sepal.Length as column name
      Warning in FUN(X[[2L]], ...) :
      Use Sepal_Width instead of Sepal.Width as column name
      Warning in FUN(X[[3L]], ...) :
      Use Petal_Length instead of Petal.Length as column name
      Warning in FUN(X[[4L]], ...) :
      Use Petal_Width instead of Petal.Width as column name
      Warning in FUN(X[[1L]], ...) :
      Use Sepal_Length instead of Sepal.Length as column name
      Warning in FUN(X[[2L]], ...) :
      Use Sepal_Width instead of Sepal.Width as column name
      Warning in FUN(X[[3L]], ...) :
      Use Petal_Length instead of Petal.Length as column name

      These are the results of having `.` in the column name. For reference, see SPARK-12191, SPARK-11976. Since it involves changing SQL, if we couldn't support that there then we should strongly consider using other dataset without `.`, eg. `cars`

      And we should update this in API doc (roxygen2 doc string), vignettes, programming guide, R code example.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            wm624 Miao Wang
            felixcheung Felix Cheung
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment