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.

Sign up for GitHub

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

Jump to bottom

Pipe: Support 'format'='tsfile' option in table model data sync sinks #14108

Merged
merged 50 commits into from
Jan 7, 2025
Merged

Pipe: Support 'format'='tsfile' option in table model data sync sinks #14108

merged 50 commits into from
Jan 7, 2025

Conversation

luoluoyuyu
Copy link
Contributor

Description

As the title said

This PR has:

  • been self-reviewed.
    • concurrent read
    • concurrent write
    • concurrent read and write
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods.
  • added or updated version, license, or notice information
  • added comments explaining the "why" and the intent of the code wherever would not be obvious
    for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold
    for code coverage.
  • added integration tests.
  • been tested in a test IoTDB cluster.
Key changed/added classes (or packages if there are too many classes) in this PR

@luoluoyuyu luoluoyuyu marked this pull request as draft November 15, 2024 16:08
@SteveYurongSu SteveYurongSu self-assigned this Nov 16, 2024
@luoluoyuyu luoluoyuyu marked this pull request as ready for review November 18, 2024 14:32
luoluoyuyu and others added 7 commits December 3, 2024 15:59
@luoluoyuyu
fix
051c1f8
@luoluoyuyu
Merge remote-tracking branch 'iotdb/master' into support-table-model-…
658e217 
…file-format

# Conflicts:
#	iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/connector/payload/evolvable/batch/PipeTabletEventTsFileBatch.java
#	iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/connector/util/TabletSortUtil.java
@luoluoyuyu
update TSFileBuilder
715c4dc
@luoluoyuyu
update PipeTabletEventSorterTest
207e56d
@luoluoyuyu
fix
806f6fb
@luoluoyuyu
Merge branch 'master' into support-table-model-file-format
802dd46
@luoluoyuyu
Merge remote-tracking branch 'origin/support-table-model-file-format'…
94c1978 
… into support-table-model-file-format
luoluoyuyu and others added 24 commits December 5, 2024 14:56
@luoluoyuyu
update OpcUaNameSpace
bf8ac82
@luoluoyuyu
update PipeTableModelTabletEventSorter
f4a7ed5
@luoluoyuyu
fix
2b6c911
@luoluoyuyu
modify code format
08f4c8a
@luoluoyuyu
Merge branch 'master' into support-table-model-file-format
ccb628f
@luoluoyuyu
add IT
a10f4e7
@luoluoyuyu
spotless
527eaa4
@luoluoyuyu
spotless
b0a112f
@luoluoyuyu
fix IT
1051ce4
@luoluoyuyu
add test case
0306ae0
@luoluoyuyu
spotless
4412268
@luoluoyuyu
fix
d4292cf
@luoluoyuyu
add test case
5b8820e
@luoluoyuyu
update PipeTableModeTsFileBuilder
9dd29ac
@luoluoyuyu
modify test case
972b172
@luoluoyuyu
modify session pool max size
b5a924f
@luoluoyuyu
close session pool
a468b79
@luoluoyuyu
modify TableModelUtils
3972607
@luoluoyuyu
Merge remote-tracking branch 'iotdb/master' into support-table-model-…
84a931b 
…file-format

# Conflicts:
#	integration-test/src/test/java/org/apache/iotdb/pipe/it/tablemodel/TableModelUtils.java
@luoluoyuyu
The receiver sorting rules are changed to be consistent with Session
f7e984d
@luoluoyuyu
update PipeTableModelTabletEventSorter
89b7d46
@SteveYurongSu
refactor
314dc52
@SteveYurongSu
Merge branch 'master' of https://github.com/apache/iotdb into pr/14108
0d1e084
@SteveYurongSu
refactor
bc97e11
@SteveYurongSu SteveYurongSu changed the title Pipe: Sink supports table model TSFile format Pipe: Support 'format'='tsfile' option in table model data sync sinks Jan 7, 2025
@SteveYurongSu SteveYurongSu merged commit f007e8a into apache:master Jan 7, 2025
34 of 35 checks passed
@luoluoyuyu luoluoyuyu deleted the support-table-model-file-format branch February 12, 2025 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SteveYurongSu SteveYurongSu SteveYurongSu approved these changes

Assignees

@SteveYurongSu SteveYurongSu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@luoluoyuyu @SteveYurongSu