From 59f26da5655418c644fbaa8d1bf24094d88000d7 Mon Sep 17 00:00:00 2001 From: Kezhu Wang Date: Tue, 26 Aug 2025 23:51:09 +0800 Subject: [PATCH] ZOOKEEPER-842: [ADDENDUM] Keep `DataTree::copyStat` for compatibility with Apache Curator `DataTree` is apparently internal to zookeeper. But we don't export public alternatives to `copyStat` and `copyStatPersisted` until now. So, let's keep them for some releases to let downstream libraries to migrate to `Stat::copyFrom`. Refs: ZOOKEEPER-842, #2305. --- .../org/apache/zookeeper/server/DataTree.java | 20 +++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/zookeeper-server/src/main/java/org/apache/zookeeper/server/DataTree.java b/zookeeper-server/src/main/java/org/apache/zookeeper/server/DataTree.java index b682ec64fb0..a9c69b48601 100644 --- a/zookeeper-server/src/main/java/org/apache/zookeeper/server/DataTree.java +++ b/zookeeper-server/src/main/java/org/apache/zookeeper/server/DataTree.java @@ -342,6 +342,26 @@ boolean isSpecialPath(String path) { || configZookeeper.equals(path); } + /** + * Use {@link StatPersisted#copyFrom(StatPersisted)} instead. + * + *

Apache Curator uses it, let's keep it for now to let them and their clients to react. + */ + @Deprecated + public static void copyStatPersisted(StatPersisted from, StatPersisted to) { + to.copyFrom(from); + } + + /** + * Use {@link Stat#copyFrom(Stat)} instead. + * + *

Apache Curator uses it, let's keep it for now to let them and their clients to react. + */ + @Deprecated + public static void copyStat(Stat from, Stat to) { + to.copyFrom(from); + } + /** * update the count/bytes of this stat data node *