flink icon indicating copy to clipboard operation
flink copied to clipboard

[FLINK-16080][docs-zh] Translate new DISTRIBUTED BY documentation into Chinese

Open JinkunLiu opened this issue 9 months ago • 7 comments

What is the purpose of the change

In Flink 1.20, FLIP-376 adds DISTRIBUTED BY. The documentation PR is here: https://github.com/apache/flink/pull/24929 and it should be translated into Chinese.

The page url is https://issues.apache.org/jira/browse/FLINK-35725

Brief change log

  • Translate new DISTRIBUTED BY documentation into Chinese
  • Fix some sql error on documentation

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): no
  • The public API, i.e., is any changed class annotated with @Public(Evolving): no
  • The serializers: no
  • The runtime per-record code paths (performance sensitive): no
  • Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
  • The S3 file system connector: no

Documentation

  • Does this pull request introduce a new feature? no
  • If yes, how is the feature documented? no

JinkunLiu avatar Feb 23 '25 13:02 JinkunLiu

CI report:

  • de3a5d366cf27bc2d5e8662d654fd5db8942c91b Azure: FAILURE
  • 19c76456ee0409a302368f09f631ea295d2e15f6 Azure: PENDING
Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

flinkbot avatar Feb 23 '25 13:02 flinkbot

@leonardBang @PatrickRen @lvyanquan @loserwang1024 @wuchong

Excuse me, would you mind taking a moment to look at this when you have time?

Thank you very much!

JinkunLiu avatar Mar 13 '25 16:03 JinkunLiu

@davidradl Hello David, sorry to bother you.

This is my first pull request, but it seems no one has reviewed it. I'm not sure what to do, could you please help me?

JinkunLiu avatar Apr 18 '25 16:04 JinkunLiu

@klion26,@RocMarshal hi,could you please help me review this commit?Thank you

JinkunLiu avatar May 17 '25 15:05 JinkunLiu

@RocMarshal Thanks for review

I'd be glad to address the heading anchor links in my existing PR. Let me make the necessary updates later.

JinkunLiu avatar May 20 '25 16:05 JinkunLiu

@RocMarshal Hello, I've added related anchors in commit de3a5d3. Could you please review my changes and let me know if this is the correct approach? Thank you for your help!

JinkunLiu avatar May 21 '25 15:05 JinkunLiu

@PatrickRen @xccui Excuse me, do you have time to take a look at this?

JinkunLiu avatar Jun 03 '25 15:06 JinkunLiu

Hey @xccui @wuchong @klion26, could you help review this PR when you have time? Thanks a lot!

JinkunLiu avatar Jul 14 '25 15:07 JinkunLiu

@jnh5y Sorry to bother you, but could you please approve the English revisions when you have a moment? Thank you!

JinkunLiu avatar Jul 14 '25 15:07 JinkunLiu

@1996fanrui @lihaosky could you help review this PR when you have time? Thanks a lot!

JinkunLiu avatar Jul 14 '25 15:07 JinkunLiu

@JinkunLiu please do not wait for my review - as I am not familiar with Chinese

davidradl avatar Sep 04 '25 15:09 davidradl

@klion26 hi,could you please help me review this commit?

JinkunLiu avatar Sep 05 '25 01:09 JinkunLiu

@huangyanyanyan Would you mind reviewing this commit?

JinkunLiu avatar Sep 05 '25 01:09 JinkunLiu

@klion26 Thanks for review!

Since this PR has been open for over six months and has received three approvals with sufficient community review, could you help merge it when convenient? I'd really appreciate your support!

Thanks for the contribution, LGTM. Let's see if there is any other comments from the community

JinkunLiu avatar Sep 05 '25 07:09 JinkunLiu

Hi @jnh5y , just checking in on this PR. Do you think it's ready to merge, or does it need one more review? Thanks!

JinkunLiu avatar Sep 10 '25 01:09 JinkunLiu

@flinkbot run azure

leonardBang avatar Sep 10 '25 02:09 leonardBang