Skip to content
This repository was archived by the owner on Jan 9, 2025. It is now read-only.

Refactoring #9

Merged
merged 6 commits into from
Jul 6, 2023
Merged

Refactoring #9

merged 6 commits into from
Jul 6, 2023

Conversation

freezemage0
Copy link
Contributor

@freezemage0 freezemage0 commented Jul 4, 2023

  • Implemented lazy fetching for QueryResponse.
  • Added PDOException wraps for Query and PreparedQuery in case if PDODriver was created with EXCEPTION PDO attribute.
  • Removed overhead in Row::getColumns() method. Added static named constructor Row::create() for convenient callback reference.
  • Implemented enumeration for DSN prefixes. Deprecated DsnBuilder constants.
  • Added missing @throws.

@freezemage0 freezemage0 self-assigned this Jul 4, 2023
Copy link
Member

@CommandString CommandString left a comment

Choose a reason for hiding this comment

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

LGTM

@freezemage0 freezemage0 force-pushed the fix/minor_refactoring branch from 0defd3c to 1be5800 Compare July 6, 2023 16:37
@freezemage0 freezemage0 merged commit d77cd96 into tnapf:main Jul 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants