-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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.
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
[ZOOKEEPER-4834] fix typos in Apache ZooKeeper #2167
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: ZhangJian He <shoothzj@gmail.com>
zookeeper-server/src/main/java/org/apache/zookeeper/server/watch/WatchManagerOptimized.java
Outdated
Show resolved
Hide resolved
@@ -51,14 +51,14 @@ public class SnapStream { | |||
|
|||
private static final Logger LOG = LoggerFactory.getLogger(SnapStream.class); | |||
|
|||
public static final String ZOOKEEPER_SHAPSHOT_STREAM_MODE = "zookeeper.snapshot.compression.method"; | |||
public static final String ZOOKEEPER_SNAPSHOT_STREAM_MODE = "zookeeper.snapshot.compression.method"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This changes public API.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I give this patch a review and find at least two public API changes. You may see if typos can be turned off inline or other workarounds.
It also changes some logs output but I think it's generally OK since we don't promise on logs.
Signed-off-by: ZhangJian He <shoothzj@gmail.com>
@tisonkun Thanks for your review. I think we can discuss later in another thread to decide public API changes(not in this PR), I revert it fisrt and add in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 to the direction.
I left some comments about some typos.
Besides above, how do we find these typos ? I saw no mvn/ci integration.
...ector/src/main/java/org/apache/zookeeper/inspector/gui/nodeviewer/NodeSelectionListener.java
Outdated
Show resolved
Hide resolved
zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Learner.java
Outdated
Show resolved
Hide resolved
zookeeper-server/src/test/java/org/apache/zookeeper/test/GetProposalFromTxnTest.java
Show resolved
Hide resolved
zookeeper-server/src/test/java/org/apache/zookeeper/test/QuorumUtilTest.java
Outdated
Show resolved
Hide resolved
Signed-off-by: ZhangJian He <shoothzj@gmail.com>
@kezhuw Thanks for your thorough review. I have address your comments.
|
@eolivelli @maoling PTAL, thanks |
See https://lists.apache.org/thread/bfxpbwscprkcyvlt64t4oyf4zr1g3dt7