Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

batch: replace the deprecated registerTempTable() to reslove the compile warning #1407

Closed
vagetablechicken opened this issue Mar 10, 2022 · 4 comments · Fixed by #1798
Closed

batch: replace the deprecated registerTempTable() to reslove the compile warning #1407

vagetablechicken opened this issue Mar 10, 2022 · 4 comments · Fixed by #1798
Labels
good first issue Good for newcomers

Comments

@vagetablechicken
Copy link
Collaborator

vagetablechicken commented Mar 10, 2022 •
edited
Loading 

[WARNING] /Users/tobe/code/4pd/OpenMLDB/java/openmldb-batch/src/main/scala/com/_4paradigm/openmldb/batch/tobe/CheckNull.scala:56: warning: method re
gisterTempTable in class Dataset is deprecated (since 2.0.0): Use createOrReplaceTempView(viewName) instead.
[WARNING]     df.registerTempTable("t1")
[WARNING]        ^
[WARNING] /Users/tobe/code/4pd/OpenMLDB/java/openmldb-batch/src/main/scala/com/_4paradigm/openmldb/batch/tobe/CheckNull.scala:57: warning: method re
gisterTempTable in class Dataset is deprecated (since 2.0.0): Use createOrReplaceTempView(viewName) instead.
[WARNING]     df.registerTempTable("t2")
[WARNING]        ^

You can reproduce the issue by compiling java modules with mvn clean package.
@vagetablechicken vagetablechicken added the good first issue Good for newcomers label Mar 10, 2022
@dl239 dl239 mentioned this issue Mar 10, 2022
Closed
@lumianph
Copy link
Collaborator

this issue belongs to our Contributor Challenges program, you can redeem your rewards after your PR is merged.
#1344

@lumianph lumianph changed the title batch: replace registerTempTable() to reslove the compile warning batch: replace the deprecated registerTempTable() to reslove the compile warning May 5, 2022
@lumianph lumianph mentioned this issue May 10, 2022
Closed
@wangxinyu666666
Copy link
Contributor

把registerTempTable替换为createOrReplaceTempView即可~

@lumianph
Copy link
Collaborator

把registerTempTable替换为createOrReplaceTempView即可~

pls bring your PR :-D @wangxinyu666666

@wangxinyu666666 wangxinyu666666 mentioned this issue May 11, 2022
Merged
@wangxinyu666666
Copy link
Contributor

把registerTempTable替换为createOrReplaceTempView即可~

pls bring your PR :-D @wangxinyu666666

#1798

@dl239 dl239 linked a pull request May 11, 2022 that will close this issue
refactor: change registerTempTable to createOrReplaceTempView #1798
Merged
@Cupid0320 Cupid0320 mentioned this issue May 11, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor: change registerTempTable to createOrReplaceTempView wangxinyu666666/OpenMLDB
3 participants
@lumianph @vagetablechicken @wangxinyu666666