adding an atime test case; groupby with dogroups (R expression) #PR4558

[DorisAmoakohene]

This test case discusses the issue reported on performing group computations, specifically when running R's C eval on each group (q7 and q8) in the db-benchmark, indicating a slowness in the implementation of the code. #4200

This is the #4558 that discusses the Cause of the Regression: #4200 (comment) which appears that the regression occurred during the evaluation of C code within these particular groups, indicating a performance issue or slowness in the implementation of the code.

The regression was fixed Regression by the addition of const int nth = getDTthreads

[github-actions]

Generated via commit 91671e4

Download link for the artifact containing the test results: ↓ atime-results.zip

Time taken to finish the standard R installation steps: 11 minutes and 55 seconds

Time taken to run atime::atime_pkg on the tests: 6 minutes and 5 seconds

[DorisAmoakohene]

@tdhock, 6288 is the right one, when I trying renaming the branch it create a new PR, I think that's the reason for the duplicate, but I am still figuring how I can rename the branch without duplicating the test

[DorisAmoakohene]

@tdhock, Kindly see the updates, so I can submit the other PR

[tdhock]

Both commits in this PR have message "Update tests.R" which may be confusing to others trying to review your PR.
Please use

git commit -m "More specific commit message which is a short description of the changes that you made"

[tdhock]

hi @Anirban166 can you please help by reviewing and giving suggestions about how to improve this PR?

[DorisAmoakohene]

Hi @tdhock and @Anirban166 could you review this for merge.

[Anirban166]

There are still pending reviews/comments that you need to push changes for

[DorisAmoakohene]

There are still pending reviews/comments that you need to push changes for

I was supposed to update the comment for the Commit id, and I have

[tdhock]

please click Files Changed tab, and read comments, then push commit(s) that address those comments.