Skip to content

debug-mode#55

Open
Kess220 wants to merge 1 commit into
mainfrom
discovery-problem-column-level
Open

debug-mode#55
Kess220 wants to merge 1 commit into
mainfrom
discovery-problem-column-level

Conversation

@Kess220
Copy link
Copy Markdown
Contributor

@Kess220 Kess220 commented Jun 6, 2025

This pull request introduces several changes across multiple files to enhance functionality, improve maintainability, and add debugging capabilities. The most significant updates include introducing a reset mechanism for column lineage, refining the column-level lineage component, and adding extensive debugging logs to the layout generation process.

Column Lineage Reset Mechanism:

  • Added a new action type RESET_COLUMN_LINEAGE in actionTypes.ts to support resetting the column lineage state.
  • Implemented a new action creator resetColumnLineage in index.ts to dispatch the reset action.
  • Updated the columnLineage reducer to handle the RESET_COLUMN_LINEAGE action, resetting the state to its initial value. [1] [2]

Column-Level Lineage Enhancements:

  • Added a new state prevDataset in ColumnLevel.tsx to detect dataset changes and trigger the resetColumnLineage action when necessary. [1] [2]
  • Updated the mapDispatchToProps and component props to include the resetColumnLineage action. [1] [2]

Debugging Improvements:

  • Introduced detailed debug logs in layout.ts to trace backend data, connected nodes, and dataset processing during the creation of Elk nodes. [1] [2] [3] [4]

Minor Code Refinements:

  • Adjusted import orders and formatting in multiple files for better readability and consistency. [1] [2] [3]
  • Fixed formatting issues and improved callback readability in ActionBar.tsx. [1] [2]

These changes collectively improve the robustness and clarity of the codebase while adding useful debugging tools for developers.

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.

1 participant