[SPARK-22430][R][Docs] Unknown tag warnings when building R docs with Roxygen 6.0.1

[rekhajoshm]

What changes were proposed in this pull request?

Removed @export tag to get rid of unknown tag warnings

How was this patch tested?

Existing tests

[SparkQA]

Test build #87040 has finished for PR 20501 at commit 7db58db.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[felixcheung]

I'm/We're aware of this and to keep diff small and manageable for backporting fixes, it is preferable to avoid broad changes like this.

We can revisit when new releases are stable down, perhaps.

[srowen]

I see this too and Roxygen 6.0.1 seems common now. Is there any value to the tags? I don't think these removals can cause much merge conflict. It can go in master right?

[felixcheung]

It was not used in the current way we have the export and a change in 6.x started warning about this.

Jenkins is not running Roxygen 6 though. There are pending tasks to upgrade. IMO it's better to update our toolset and validate the doc output is correct officially and then remove these tags for the warnings. The warnings are harmless.

Historically a lot of changes get backported around a new releases (or RCs of releases) and it was a pain to hand merge every single commit.

[rekhajoshm]

Ack. thanks for the update @felixcheung @srowen Closing this.

[felixcheung]

@rekhajoshm feel free to follow up after we are through with 2.3.0, thanks!

[srowen]

@rekhajoshm @felixcheung since 2.3.0 was just finished, OK to merge?

[felixcheung]

can you reopen this PR and add [R] in the title?

[felixcheung]

ping @rekhajoshm ?

[rekhajoshm]

done @felixcheung thanks

[SparkQA]

Test build #87949 has finished for PR 20501 at commit 7db58db.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[felixcheung]

merged to master