You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "vibhatha (via GitHub)" <gi...@apache.org> on 2024/05/16 01:18:08 UTC
[I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
vibhatha opened a new issue, #41680:
URL: https://github.com/apache/arrow/issues/41680
### Describe the enhancement requested
At the moment, test cases are using an older API version of Junit, and this should be upgraded to use the latest API.
### Component(s)
Java
--
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: issues-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "vibhatha (via GitHub)" <gi...@apache.org>.
vibhatha commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2172108098
@llama90 Thank you for updating it here. I will make sure to update the list. I am out today, will be a bit slow on reviews.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "llama90 (via GitHub)" <gi...@apache.org>.
llama90 commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2156584902
@vibhatha I have verified the `c` module using only JUnit 5. Please cross-check when you have free time. Thank you.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "vibhatha (via GitHub)" <gi...@apache.org>.
vibhatha commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2163278178
@llama90 Done! Thanks for looking into 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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "vibhatha (via GitHub)" <gi...@apache.org>.
vibhatha commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2156908362
@llama90 You are correct, we don't need to work on `c` module. It is already in JUNIT5. Thanks for verifying 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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "vibhatha (via GitHub)" <gi...@apache.org>.
vibhatha commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2156604362
> @vibhatha I have verified the `c` module using only JUnit 5. Please cross-check when you have free time. Thank you.
>
>
>
>
@llama90 thank you for checking this one. I will verify. I will update tomorrow. Really appreciate your support on this upgrade.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "llama90 (via GitHub)" <gi...@apache.org>.
llama90 commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2171748135
@vibhatha I attached the remaining issues for updating the contents.
- https://github.com/apache/arrow/issues/42045
- https://github.com/apache/arrow/issues/42162
- https://github.com/apache/arrow/issues/42164
- https://github.com/apache/arrow/issues/42165
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "vibhatha (via GitHub)" <gi...@apache.org>.
vibhatha commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2174878610
@lidavidm I will work on that. Definitely.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "vibhatha (via GitHub)" <gi...@apache.org>.
vibhatha commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2174881145
@lidavidm issue created: https://github.com/apache/arrow/issues/42193
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "llama90 (via GitHub)" <gi...@apache.org>.
llama90 commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2172111532
No problem! Have a nice day.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "llama90 (via GitHub)" <gi...@apache.org>.
llama90 commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2158580739
@vibhatha I think I will start after merging the spotless changes for each module to avoid conflicts.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "vibhatha (via GitHub)" <gi...@apache.org>.
vibhatha commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2158594767
@llama90 that's good idea.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "llama90 (via GitHub)" <gi...@apache.org>.
llama90 commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2163203243
@vibhatha Additionally, the `performance` module does not include tests, so there is no need to assign tasks for it.
Please cross-check when you have free time. Thank you.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Java] Update Unit Tests to use `org.junit.jupiter.api` [arrow]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on issue #41680:
URL: https://github.com/apache/arrow/issues/41680#issuecomment-2174701132
@vibhatha after this is done, we also need to remove all JUnit 4 dependencies and preferably exclude them so we don't accidentally re-take a dependency on JUnit 4
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org