You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sedona.apache.org by GitBox <gi...@apache.org> on 2022/12/02 01:06:06 UTC
[GitHub] [incubator-sedona] jiayuasu opened a new pull request, #721: [DOCS] Fix the R test that still has DBI call
jiayuasu opened a new pull request, #721:
URL: https://github.com/apache/incubator-sedona/pull/721
## Did you read the Contributor Guide?
- Yes, I have read [Contributor Rules](https://sedona.apache.org/community/rule/) and [Contributor Development Guide](https://sedona.apache.org/community/develop/)
## Is this PR related to a JIRA ticket?
- No, this is a documentation update. The PR name follows the format `[DOCS] my subject`.
## What changes were proposed in this PR?
## How was this patch tested?
## Did this PR include necessary documentation updates?
- No, this PR does not affect any public API so no need to change the docs.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@sedona.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-sedona] jiayuasu commented on pull request #721: [DOCS] Fix the R test that still has DBI call
Posted by GitBox <gi...@apache.org>.
jiayuasu commented on PR #721:
URL: https://github.com/apache/incubator-sedona/pull/721#issuecomment-1345389793
@yitao-li Dear Yitao, could you please help me remove the DBI call in the test? My PR didn't work for this...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@sedona.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-sedona] yitao-li commented on pull request #721: [DOCS] Fix the R test that still has DBI call
Posted by GitBox <gi...@apache.org>.
yitao-li commented on PR #721:
URL: https://github.com/apache/incubator-sedona/pull/721#issuecomment-1346867576
@jiayuasu I think https://github.com/apache/incubator-sedona/pull/731 should fix it.
The CI run for that PR was cancelled after some unrelated test failure though, but if I run my version of `sdf_sql(...) %>% collect()` locally, I do get the expected result, so, I think it should work.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@sedona.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-sedona] jiayuasu closed pull request #721: [DOCS] Fix the R test that still has DBI call
Posted by GitBox <gi...@apache.org>.
jiayuasu closed pull request #721: [DOCS] Fix the R test that still has DBI call
URL: https://github.com/apache/incubator-sedona/pull/721
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@sedona.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-sedona] yitao-li commented on pull request #721: [DOCS] Fix the R test that still has DBI call
Posted by GitBox <gi...@apache.org>.
yitao-li commented on PR #721:
URL: https://github.com/apache/incubator-sedona/pull/721#issuecomment-1345458994
Uh just noticed the build failures... ok I'll take a look soon
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@sedona.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org