Skip to content

[IOTDB-2671][metrics] Improve the implementation of iotdbReporter#5338

Merged
SteveYurongSu merged 23 commits intoapache:masterfrom
SpriCoder:improve/iotdb_reporter_impl
Mar 27, 2022
Merged

[IOTDB-2671][metrics] Improve the implementation of iotdbReporter#5338
SteveYurongSu merged 23 commits intoapache:masterfrom
SpriCoder:improve/iotdb_reporter_impl

Conversation

@SpriCoder
Copy link
Contributor

@SpriCoder
Copy link
Contributor Author

@SteveYurongSu PTAL!

@SteveYurongSu
Copy link
Member

@SteveYurongSu PTAL!

Hi, great! Please resolve the conflict :D

stormbroken added 2 commits March 25, 2022 15:23
…orter_impl

# Conflicts:
#	metrics/dropwizard-metrics/src/main/java/org/apache/iotdb/metrics/dropwizard/MetricName.java
#	metrics/dropwizard-metrics/src/main/java/org/apache/iotdb/metrics/dropwizard/reporter/DropwizardMetricsExporter.java
@SpriCoder
Copy link
Contributor Author

@SteveYurongSu PTAL!

Hi, great! Please resolve the conflict :D

Ok, I resolve the conflict.

@jun0315
Copy link
Contributor

jun0315 commented Mar 26, 2022

LGTM

Copy link
Member

@SteveYurongSu SteveYurongSu left a comment

Choose a reason for hiding this comment

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

LGTM.

Maybe we can use SessionPool instead of Session for fault tolerance. This can be a new ticket for u.

@SteveYurongSu SteveYurongSu merged commit c6e47b9 into apache:master Mar 27, 2022
SpriCoder added a commit to SpriCoder/iotdb that referenced this pull request Mar 28, 2022
SpriCoder pushed a commit to SpriCoder/iotdb that referenced this pull request Mar 28, 2022
SteveYurongSu pushed a commit that referenced this pull request Apr 2, 2022
…pdate the implementation of IoTDBReporter and fix NPE error. (#5358)

* [IOTDB-2777][metrics] Fix NPE and sample frequency (#5304)

Co-authored-by: 还真 <huanzhen.gy@alibaba-inc.com>

(cherry picked from commit b9b32e1)

* [IOTDB-2762][metrics] Fix inconsistent name between dropwizard implementation and micrometer implementation (#5301)

(cherry picked from commit 8f9d829)

* [IOTDB-2671][metrics] Improve the implementation of iotdbReporter (#5338)

(cherry picked from commit c6e47b9)

* [IOTDB-2786][Dependency] remove iotdb-session dependent on iotdb-server(test) (#5319)

(cherry picked from commit 3dca44a)

* [IOTDB-2671][metrics] Improve the implementation of iotdbReporter (#5338)

(cherry picked from commit c6e47b9)

Co-authored-by: xinzhongtianxia <453935989@qq.com>
xinzhongtianxia pushed a commit to xinzhongtianxia/iotdb that referenced this pull request Apr 5, 2022
…er_meta

* remotes/upstream/master:
  Rename SchemaEngine to LocalSchemaProcessor and Rename SchemaRegionManager to SchemaEngine (apache#5359)
  Add SchemaRegion Interfaces to SchemaEngine (apache#5349)
  [IOTDB-2803] adapt metadata write operation to plannode (apache#5344)
  [IOTDB-2679] Support logical operators in select clauses (apache#5273)
  [IOTDB-2671][metrics] Improve the implementation of iotdbReporter (apache#5338)

# Conflicts:
#	server/src/main/java/org/apache/iotdb/db/mpp/sql/planner/LogicalPlanner.java
#	server/src/test/java/org/apache/iotdb/db/mpp/sql/plan/LogicalPlannerTest.java
@SpriCoder SpriCoder deleted the improve/iotdb_reporter_impl branch May 17, 2022 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants