Skip to content

[ARC-19] fix: OOM render issue #172

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

Merged
merged 18 commits into from
Jul 11, 2025
Merged

[ARC-19] fix: OOM render issue #172

merged 18 commits into from
Jul 11, 2025

Conversation

BornPsych
Copy link
Collaborator

  1. Date Picker Fix
    • Resolved an issue with the manual entry in the React Date Picker.
    • The component is now editable manually without rendering glitches.

  2. Logs Rendering for GCD calculation
    • Fixed the issue where logs were not being rendered for GCD calculationI.
    • Also added the count to check how many GCD task were send for calculation

  3. Database Query Optimization - Performance improved from 3–6 seconds (avg. ~3s) to sub-millisecond average response times.
    • Indexing - This dramatically improved performance on frequently queried fields used in synchronous calls.
    • Conditional Query Execution - Moving non-critical queries under conditional blocks (if statements) for deferred/lazy evaluation.
    • Query Refactor for Timestamp Lookup - Previously executed sequentially, now handled using Promise.all() for parallel execution.
    • Raw SQL Execution & Conflict Handling - Leveraged PostgreSQL’s ON CONFLICT to eliminate unnecessary DB round-trips for existence checks

@zkemailverify zkemailverify temporarily deployed to fix/OOM-render-issue - archive-mirror PR #172 July 2, 2025 19:33 — with Render Destroyed
@BornPsych BornPsych changed the title fix: OOM render issue [ARC-19] fix: OOM render issue Jul 2, 2025
@BornPsych BornPsych requested a review from foolo July 2, 2025 19:46
Copy link
Contributor

@foolo foolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

review done, nice speed-up! :)

@zkemailverify zkemailverify temporarily deployed to fix/OOM-render-issue - archive-mirror PR #172 July 7, 2025 17:15 — with Render Destroyed
@zkemailverify zkemailverify temporarily deployed to fix/OOM-render-issue - archive-mirror PR #172 July 7, 2025 17:16 — with Render Destroyed
@Divide-By-0
Copy link
Member

Divide-By-0 commented Jul 10, 2025

Lgtm, feel free to merge, I think in a future PR we can try to profile and understand where the main slowdowns are coming from

@BornPsych BornPsych merged commit b2a0ded into main Jul 11, 2025
4 checks passed
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.

4 participants