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

Chore performance improvements tech debt 1 #3607

Conversation

xoscar
Copy link
Collaborator

@xoscar xoscar commented Feb 6, 2024

This PR fixes several of the todos for the trace view, it also improves the way we are rendering the DAG in case a big list of spans are in display.

Changes

  • Adds loading spinner for test run
  • Adds loading spinner for DAG rendering
  • Improves performance and data structures

Fixes

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

Copy link

vercel bot commented Feb 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tracetest ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 8, 2024 4:23pm

@xoscar xoscar changed the base branch from main to chore-performance-improvements February 8, 2024 15:21
Copy link
Contributor

@jorgeepc jorgeepc left a comment

Choose a reason for hiding this comment

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

Looking good. A couple of comments.

@@ -9,7 +9,7 @@ import useSelectAsCurrent from '../../../hooks/useSelectAsCurrent';

interface IProps extends NodeProps<INodeDataSpan> {}

const TestSpanNode = ({data, id, selected}: IProps) => {
const TestSpanNode = ({data, id, selected, ...props}: IProps) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

🧼 props not used

return DAGService.getEdgesAndNodes(nodesDatum);
}

export const getShouldShowDAG = (spanCount: number): boolean => spanCount <= 200;
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we use the constant here?

@@ -87,3 +87,12 @@ export const getParsedURL = (rawUrl: string): URL => {

return new URL(rawUrl);
};

export const withLowPriority =
Copy link
Contributor

Choose a reason for hiding this comment

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

🪄

@xoscar xoscar merged commit 9a34b5e into chore-performance-improvements Feb 8, 2024
39 checks passed
@xoscar xoscar deleted the chore-performance-improvements-tech-debt-1 branch February 8, 2024 19:45
xoscar added a commit that referenced this pull request Feb 13, 2024
* chore(frontend): indentifying performance improvements

* chore(frontend): identify performance improvements

* feat(frontend): hide dag view for big traces (#3606)

* Chore performance improvements tech debt 1 (#3607)

* adding async functionality

* chore(frontend): DAG improvements

* chore(frontend): DAG improvements

* chore(frontend): fixing spinner styles

* chore(frontend): fixing spinner styles

* feat(frontend): fixing tests

* chore(frontend): fixing spinner styles

* chore(frontend): FE Improvements for the Test View (#3613)

* adding async functionality

* chore(frontend): DAG improvements

* chore(frontend): DAG improvements

* chore(frontend): fixing spinner styles

* chore(frontend): fixing spinner styles

* feat(frontend): fixing tests

* chore(frontend): fixing spinner styles

* chore(frontend): FE Improvements for the Test View

* chore(frontend): reverting editor changes

* chore(frontend): Adding memoization for useComplete hook

* chore(frontend): Adding Search Service usage

* chore(frontend): cleanup

* feat(frontend): implement virtual list for timeline view (#3617)

* feat(frontend): implement virtual list for timeline view

* remove prop

* add header and collapse

* feat: add timeline view connectors (#3623)

* feat(frontend): Implemeting new Timeline for the Test Page (#3627)

* feat(frontend): Implemeting new Timeline for the Test Page

* feat(frontend): Fixing tests

* feat(frontend): Fixing tests

* Chore performance improvements span search (#3629)

* feat(frontend): Implemeting new Timeline for the Test Page

* feat(frontend): Implementing span search for analyzer and test views

* feat(frontend): read improvements

* feat(frontend): adding single line input component

* feat(frontend): updating analyzer styles

* feat(frontend): Fixing tests

* feat(frontend): Fixing tests

---------

Co-authored-by: Jorge Padilla <jorge.esteban.padilla@gmail.com>
mathnogueira added a commit that referenced this pull request Feb 13, 2024
* start dashboard if agent mode is set as dashboard

* update environment and collector statistics

* feat: update list of tests based on test runs received by the agent

* update test run counter

* make dashboard as part of the desktop option list

* open test run in browser

* fix panic

* make it possible to exit and go back to dashboard

* remove dashboard mode from CLI

* Chore performance improvements (#3616)

* chore(frontend): indentifying performance improvements

* chore(frontend): identify performance improvements

* feat(frontend): hide dag view for big traces (#3606)

* Chore performance improvements tech debt 1 (#3607)

* adding async functionality

* chore(frontend): DAG improvements

* chore(frontend): DAG improvements

* chore(frontend): fixing spinner styles

* chore(frontend): fixing spinner styles

* feat(frontend): fixing tests

* chore(frontend): fixing spinner styles

* chore(frontend): FE Improvements for the Test View (#3613)

* adding async functionality

* chore(frontend): DAG improvements

* chore(frontend): DAG improvements

* chore(frontend): fixing spinner styles

* chore(frontend): fixing spinner styles

* feat(frontend): fixing tests

* chore(frontend): fixing spinner styles

* chore(frontend): FE Improvements for the Test View

* chore(frontend): reverting editor changes

* chore(frontend): Adding memoization for useComplete hook

* chore(frontend): Adding Search Service usage

* chore(frontend): cleanup

* feat(frontend): implement virtual list for timeline view (#3617)

* feat(frontend): implement virtual list for timeline view

* remove prop

* add header and collapse

* feat: add timeline view connectors (#3623)

* feat(frontend): Implemeting new Timeline for the Test Page (#3627)

* feat(frontend): Implemeting new Timeline for the Test Page

* feat(frontend): Fixing tests

* feat(frontend): Fixing tests

* Chore performance improvements span search (#3629)

* feat(frontend): Implemeting new Timeline for the Test Page

* feat(frontend): Implementing span search for analyzer and test views

* feat(frontend): read improvements

* feat(frontend): adding single line input component

* feat(frontend): updating analyzer styles

* feat(frontend): Fixing tests

* feat(frontend): Fixing tests

---------

Co-authored-by: Jorge Padilla <jorge.esteban.padilla@gmail.com>

* disable logger in dashboard mode

* keep old log level

* add comment about why we are disabling logs in dashboard mode

---------

Co-authored-by: Oscar Reyes <oscar-rreyes1@hotmail.com>
Co-authored-by: Jorge Padilla <jorge.esteban.padilla@gmail.com>
mathnogueira added a commit that referenced this pull request Feb 13, 2024
* feat: agent dashboard header layout (#3592)

* add initial dashboard code

* implement banner for messages in header

* commit rest of files

* fix build and set logo static

* feat: agent UI test run list (#3619)

* adapt layout to match @olha23's design

* add test run list

* docs: comment some layout explanation

* feat: agent dashboard start (#3631)

* start dashboard if agent mode is set as dashboard

* update environment and collector statistics

* feat: update list of tests based on test runs received by the agent

* update test run counter

* make dashboard as part of the desktop option list

* open test run in browser

* fix panic

* make it possible to exit and go back to dashboard

* remove dashboard mode from CLI

* Chore performance improvements (#3616)

* chore(frontend): indentifying performance improvements

* chore(frontend): identify performance improvements

* feat(frontend): hide dag view for big traces (#3606)

* Chore performance improvements tech debt 1 (#3607)

* adding async functionality

* chore(frontend): DAG improvements

* chore(frontend): DAG improvements

* chore(frontend): fixing spinner styles

* chore(frontend): fixing spinner styles

* feat(frontend): fixing tests

* chore(frontend): fixing spinner styles

* chore(frontend): FE Improvements for the Test View (#3613)

* adding async functionality

* chore(frontend): DAG improvements

* chore(frontend): DAG improvements

* chore(frontend): fixing spinner styles

* chore(frontend): fixing spinner styles

* feat(frontend): fixing tests

* chore(frontend): fixing spinner styles

* chore(frontend): FE Improvements for the Test View

* chore(frontend): reverting editor changes

* chore(frontend): Adding memoization for useComplete hook

* chore(frontend): Adding Search Service usage

* chore(frontend): cleanup

* feat(frontend): implement virtual list for timeline view (#3617)

* feat(frontend): implement virtual list for timeline view

* remove prop

* add header and collapse

* feat: add timeline view connectors (#3623)

* feat(frontend): Implemeting new Timeline for the Test Page (#3627)

* feat(frontend): Implemeting new Timeline for the Test Page

* feat(frontend): Fixing tests

* feat(frontend): Fixing tests

* Chore performance improvements span search (#3629)

* feat(frontend): Implemeting new Timeline for the Test Page

* feat(frontend): Implementing span search for analyzer and test views

* feat(frontend): read improvements

* feat(frontend): adding single line input component

* feat(frontend): updating analyzer styles

* feat(frontend): Fixing tests

* feat(frontend): Fixing tests

---------

Co-authored-by: Jorge Padilla <jorge.esteban.padilla@gmail.com>

* disable logger in dashboard mode

* keep old log level

* add comment about why we are disabling logs in dashboard mode

---------

Co-authored-by: Oscar Reyes <oscar-rreyes1@hotmail.com>
Co-authored-by: Jorge Padilla <jorge.esteban.padilla@gmail.com>

---------

Co-authored-by: Oscar Reyes <oscar-rreyes1@hotmail.com>
Co-authored-by: Jorge Padilla <jorge.esteban.padilla@gmail.com>
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

2 participants