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

Separate table/tree model database & Added SchemaRegionGroupNum / dataRegionGroupNum in table databases details & Optimized the message for table flush operation with database #14398

Merged
merged 108 commits into from
Dec 20, 2024
Merged

Separate table/tree model database & Added SchemaRegionGroupNum / dataRegionGroupNum in table databases details & Optimized the message for table flush operation with database #14398

merged 108 commits into from
Dec 20, 2024

Conversation

Caideyipi
Copy link
Collaborator

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

@Caideyipi Caideyipi changed the title Separate table/tree model database Separate table/tree model database & Added SchemaRegionGroupNum|&dataRegionGroupNum in table databases details Dec 12, 2024
@Caideyipi Caideyipi changed the title Separate table/tree model database & Added SchemaRegionGroupNum|&dataRegionGroupNum in table databases details Separate table/tree model database & Added SchemaRegionGroupNum / dataRegionGroupNum in table databases details Dec 12, 2024
@Caideyipi Caideyipi changed the title Separate table/tree model database & Added SchemaRegionGroupNum / dataRegionGroupNum in table databases details & Fixed the message and database semantic for table flush operation with database Separate table/tree model database & Added SchemaRegionGroupNum / dataRegionGroupNum in table databases details & Fixed the message and database semantic for table flush operation with database / Fixed the bug that partitionCache always miss for partition without deviceIDs Dec 17, 2024
@Caideyipi Caideyipi changed the title Separate table/tree model database & Added SchemaRegionGroupNum / dataRegionGroupNum in table databases details & Fixed the message and database semantic for table flush operation with database / Fixed the bug that partitionCache always miss for partition without deviceIDs Separate table/tree model database & Added SchemaRegionGroupNum / dataRegionGroupNum in table databases details & Fixed the message and database semantic for table flush operation with database Dec 17, 2024
@Caideyipi Caideyipi changed the title Separate table/tree model database & Added SchemaRegionGroupNum / dataRegionGroupNum in table databases details & Fixed the message and database semantic for table flush operation with database Separate table/tree model database & Added SchemaRegionGroupNum / dataRegionGroupNum in table databases details & Optimized the message for table flush operation with database Dec 19, 2024
JackieTien97
JackieTien97 requested changes Dec 19, 2024
View reviewed changes
...onfignode/src/main/java/org/apache/iotdb/confignode/manager/schema/ClusterSchemaManager.java Outdated Show resolved Hide resolved
...nfignode/src/main/java/org/apache/iotdb/confignode/persistence/schema/ClusterSchemaInfo.java Show resolved Hide resolved
...nfignode/src/main/java/org/apache/iotdb/confignode/persistence/schema/ClusterSchemaInfo.java Show resolved Hide resolved
...nfignode/src/main/java/org/apache/iotdb/confignode/persistence/schema/ClusterSchemaInfo.java Show resolved Hide resolved
iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/event/common/PipeInsertionEvent.java Show resolved Hide resolved
iotdb-core/datanode/src/test/java/org/apache/iotdb/db/utils/SchemaRegionSnapshotParserTest.java Show resolved Hide resolved
@Caideyipi
Update ClusterSchemaInfo.java
cd78a61
@Caideyipi
Fix
235bb0b
@Caideyipi
Merge remote-tracking branch 'upstream/master' into separate-db
44c3b01 
# Conflicts:
#	integration-test/src/test/java/org/apache/iotdb/relational/it/schema/IoTDBDatabaseIT.java
#	iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/execution/config/metadata/relational/ShowDBTask.java
#	iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/metadata/TableMetadataImpl.java
#	iotdb-core/datanode/src/test/java/org/apache/iotdb/db/queryengine/plan/relational/analyzer/TestMatadata.java
@Caideyipi
Refactor
cd6f3da
@Caideyipi
Update InformationSchema.java
e719594
@Caideyipi
Update IoTDBDatabaseIT.java
7503b07
@Caideyipi
Merge1
b167145
@Caideyipi
MERGE2
4e5a581
@Caideyipi
Fix
8653a0f
@Caideyipi
Update IoTDBDatabaseIT.java
ea1e7af
@Caideyipi
Merge branch 'master' of https://github.com/apache/iotdb into separat…
c4675f6 
…e-db
@Caideyipi
Update IoTDBDatabaseIT.java
840faaf
@JackieTien97 JackieTien97 merged commit aee90fb into apache:master Dec 20, 2024
33 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JackieTien97 JackieTien97 JackieTien97 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Caideyipi @JackieTien97