Skip to content

[To dev/1.3] Fix SessionDataSet sync-over-async deadlock and migrate to 4-arg RowRecord constructor#49

Merged
HTHou merged 1 commit intodev/1.3from
fix/dev-1.3
Mar 13, 2026
Merged

[To dev/1.3] Fix SessionDataSet sync-over-async deadlock and migrate to 4-arg RowRecord constructor#49
HTHou merged 1 commit intodev/1.3from
fix/dev-1.3

Conversation

@CritasWang
Copy link
Contributor

@CritasWang CritasWang commented Mar 11, 2026

link #48

  • Add HasNextAsync() and FetchResultsAsync() to SessionDataSet to avoid sync-over-async deadlock in FetchResults()
  • Mark HasNext() as [Obsolete] with guidance to use HasNextAsync()
  • Update ConstructOneRow() to use 4-arg RowRecord constructor with dataTypes
  • Fix IoTDBCommand.BindParameters to use 4-arg RowRecord constructor
  • Add #pragma warning suppress in IoTDBDataReader for required sync calls
  • Update SessionPool.CheckTimeSeriesExistsAsync to use HasNextAsync()
  • Migrate all sample code to use await HasNextAsync() and 4-arg RowRecord

…ecord constructor

- Add HasNextAsync() and FetchResultsAsync() to SessionDataSet to avoid
  sync-over-async deadlock in FetchResults()
- Mark HasNext() as [Obsolete] with guidance to use HasNextAsync()
- Update ConstructOneRow() to use 4-arg RowRecord constructor with dataTypes
- Fix IoTDBCommand.BindParameters to use 4-arg RowRecord constructor
- Add #pragma warning suppress in IoTDBDataReader for required sync calls
- Update SessionPool.CheckTimeSeriesExistsAsync to use HasNextAsync()
- Migrate all sample code to use await HasNextAsync() and 4-arg RowRecord
@CritasWang CritasWang changed the title Fix SessionDataSet sync-over-async deadlock and migrate to 4-arg RowRecord constructor [To dev/1.3] Fix SessionDataSet sync-over-async deadlock and migrate to 4-arg RowRecord constructor Mar 11, 2026
@CritasWang CritasWang requested a review from HTHou March 13, 2026 04:51
@HTHou HTHou merged commit 1a5317e into dev/1.3 Mar 13, 2026
5 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.

2 participants