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.

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

ECSqlStatement toRow API binding #6840

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

akshay05n
Copy link
Contributor

imodel-native: iTwin/imodel-native#786

@akshay05n
Copy link
Contributor Author

Performance Improvements:

@khanaffan @ColinKerr The following table showcases the performance improvements achieved through this change. The measurements were conducted on my local machine by running the test 6-7 times and averaging the results. It's important to note that these numbers can vary slightly due to other applications running on the machine during the tests.

ElementClassName Element Count Before (Avg Time in sec) After (Avg Time in sec) Improvement
PerfElement2d 10000 3.2661 0.60725 81.31%
PerfElement2d 100000 29.3028 5.90163 79.84%
PerfElement2dSub1 10000 3.8125 0.6822 82.12%
PerfElement2dSub1 100000 37.7555 6.17383 83.65%
PerfElement2dSub2 10000 4.8337 0.68654 85.80%
PerfElement2dSub2 100000 46.6497 6.47896 86.11%
PerfElement2dSub3 10000 6.0925 0.72204 88.15%
PerfElement2dSub3 100000 60.2998 7.46575 87.62%
PerfElement3d 10000 3.3252 0.6332 80.96%
PerfElement3d 100000 32.9899 5.8268 82.34%
PerfElement3dSub1 10000 3.9245 0.638 83.74%
PerfElement3dSub1 100000 38.1947 6.3721 83.32%
PerfElement3dSub2 10000 4.3661 0.6871 84.27%
PerfElement3dSub2 100000 42.4498 6.5449 84.58%
PerfElement3dSub3 10000 5.3051 0.7357 86.13%
PerfElement3dSub3 100000 52.2635 7.1111 86.39%

@akshay05n akshay05n marked this pull request as ready for review July 11, 2024 10:40
@akshay05n akshay05n requested a review from a team as a code owner July 11, 2024 10:40
@akshay05n akshay05n requested a review from a team as a code owner July 15, 2024 07:09
@akshay05n akshay05n force-pushed the aks/ecsql-to-row-api-integration branch from b82746e to 7a66894 Compare July 18, 2024 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants