Skip to content

typos/grammar#22

Closed
luss wants to merge 1 commit intoIvorySQL:masterfrom
pgsql-io:master
Closed

typos/grammar#22
luss wants to merge 1 commit intoIvorySQL:masterfrom
pgsql-io:master

Conversation

@luss
Copy link
Copy Markdown

@luss luss commented Dec 17, 2021

No description provided.

@rasifr rasifr changed the base branch from master to IVORY_REL_1_STABLE December 29, 2021 14:47
@rasifr rasifr changed the base branch from IVORY_REL_1_STABLE to master December 29, 2021 14:48
@rasifr
Copy link
Copy Markdown

rasifr commented Dec 29, 2021

applied and committed.

@rasifr rasifr closed this Dec 29, 2021
gaoxueyu pushed a commit that referenced this pull request Jan 9, 2026
Address CodeRabbit review feedback:
- Change "full Oracle compatibility" to "high Oracle compatibility"
- Clarify that re-ENABLE clearing buffer is IvorySQL behavior (not Oracle)
- Clarify that buffer size range is IvorySQL-enforced (stricter than Oracle)
- Reference GitHub issues #22 and #26 for tracking differences
gaoxueyu pushed a commit that referenced this pull request Jan 9, 2026
- ENABLE(NULL) now uses max buffer (1000000) instead of INT_MAX
- Add named constants for buffer size limits and line allocation
- Add XACT_EVENT_PARALLEL_COMMIT/ABORT handling
- Remove references to external issue trackers (#22, #26)
- Update test comments to reflect NULL buffer behavior
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.

3 participants