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.

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

[website][upgrade]feat: docs migration - version-2.7.2 Pulsar IO #12394

Merged
merged 16 commits into from
Oct 28, 2021

Conversation

zeo1995
Copy link
Contributor

@zeo1995 zeo1995 commented Oct 17, 2021

Master Issue: #11766

Motivation

website upgrade

Modifications

Click to Preview

Snapshot

image

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

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

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

  • Added integration tests for end-to-end deployment with large payloads (10MB)
  • Extended integration test for recovery after broker failure

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

If yes was chosen, please highlight the changes

  • Dependencies (does it add or upgrade a dependency): (yes / no)
  • The public API: (yes / no)
  • The schema: (yes / no / don't know)
  • The default values of configurations: (yes / no)
  • The wire protocol: (yes / no)
  • The rest endpoints: (yes / no)
  • The admin cli options: (yes / no)
  • Anything that affects deployment: (yes / no / don't know)

Documentation

For contributor

For this PR, do we need to update docs?

  • If yes, please update docs or create a follow-up issue if you need help.

  • If no, please explain why.

For committer

For this PR, do we need to update docs?

  • If yes,

    • if you update docs in this PR, label this PR with the doc label.

    • if you plan to update docs later, label this PR with the doc-required label.

    • if you need help on updating docs, create a follow-up issue with the doc-required label.

  • If no, label this PR with the no-need-doc label and explain why.

@Anonymitaet Anonymitaet requested a review from tuteng October 18, 2021 03:23
@Anonymitaet Anonymitaet added component/website doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. labels Oct 18, 2021
@Anonymitaet Anonymitaet added this to the 2.10.0 milestone Oct 18, 2021
@Anonymitaet
Copy link
Member

Anonymitaet commented Oct 18, 2021

@zeo1995 :

Website: https://pulsar.apache.org/docs/en/next/io-quickstart/

Display issues:

#1
image

#2
image

#3
image

#4
image

#5
image

---dividing line---

@urfreespace 2.8.0 has the above issues, could you please help check? Thanks

@Anonymitaet
Copy link
Member

Anonymitaet commented Oct 18, 2021

@zeo1995
Website: https://deploy-preview-2--distracted-lumiere-83f5b7.netlify.app/docs/2.7.2/io-use

Display issues:

image

image

image

---dividing line---

@urfreespace 2.8.0 has the above issues, could you please help check? Thanks

@Anonymitaet
Copy link
Member

@zeo1995 website: https://deploy-preview-2--distracted-lumiere-83f5b7.netlify.app/docs/2.7.2/io-debug

display issues:

image

image

image

---dividing line---

@urfreespace 2.8.0 and 2.7.3 have the above issues, could you please help check? Thanks

@urfreespace
Copy link
Member

OK, I'll take a look. @Anonymitaet

@urfreespace
Copy link
Member

@zeo1995 please update your repo scripts, and re-run the script for your pr, submit your change after you make sure all these issues had been resolved.

Yan Zhang added 3 commits October 20, 2021 23:40
* up/master:
  [pulsar-java-client] Auto-recovery after exception like out of direct memory (apache#12170)
  Allow to config pulsar client allocator out of memory policy (apache#12200)
  [Transaction] Fix bugs, Exception thrower by TB::appendBufferToTxn must be ManagedLedgerException.  (apache#12376)
  Bumped version to 2.10.0-SNAPSHOT (apache#12285)
  [docs][Website] add docs of broker entry metadata (apache#12404)
  [C++] Use weak ref to ClientConnection for timeout task (apache#12409)
  fix windows test path probleam (apache#12398)
  [website][upgrade]feat: home page (apache#12383)
  [docs] [ISSUE 11526] Update the description of `topic` (apache#12375)
  [Docs] Add document label check robot (apache#12371)
  [Admin] Get schema validation enforce add applied. (apache#12349)
  [Doc] add scope explanations (apache#12380)
  Fix java doc for MultipleListenerValidator (apache#12389)
….2-chapter-4

* website/branch-2.7.2-chapter-3:
  Update the nesting of forms in the document
  [pulsar-java-client] Auto-recovery after exception like out of direct memory (apache#12170)
  Allow to config pulsar client allocator out of memory policy (apache#12200)
  [Transaction] Fix bugs, Exception thrower by TB::appendBufferToTxn must be ManagedLedgerException.  (apache#12376)
  Bumped version to 2.10.0-SNAPSHOT (apache#12285)
  [docs][Website] add docs of broker entry metadata (apache#12404)
  [C++] Use weak ref to ClientConnection for timeout task (apache#12409)
  fix windows test path probleam (apache#12398)
  [website][upgrade]feat: home page (apache#12383)
  [docs] [ISSUE 11526] Update the description of `topic` (apache#12375)
  [Docs] Add document label check robot (apache#12371)
  [Admin] Get schema validation enforce add applied. (apache#12349)
  [Doc] add scope explanations (apache#12380)
  Fix java doc for MultipleListenerValidator (apache#12389)
@zeo1995 zeo1995 changed the title Website/branch 2.7.2 chapter 4 [website][upgrade]feat: docs migration - version-2.7.2 Pulsar IO Oct 21, 2021
@zeo1995
Copy link
Contributor Author

zeo1995 commented Oct 21, 2021

@zeo1995 please update your repo scripts, and re-run the script for your pr, submit your change after you make sure all these issues had been resolved.

@urfreespace Document styles have been updated

* up/master: (46 commits)
  [website][upgrade]feat: docs migration - version-2.7.2 Pulsar Schema (apache#12393)
  [docs] io-develop, fix broken link (apache#12414)
  docs(function): fix incorrect classname in python runtime sample (apache#12476)
  Remove redundant null check for getInternalListener (apache#12474)
  Fix the retry topic's `REAL_TOPIC` & `ORIGIN_MESSAGE_ID` property should not be modified once it has been written. (apache#12451)
  [cli] Fix output format of string by pulsar-admin command (apache#11878)
  fix the race of delete subscription and delete topic (apache#12240)
  fix influxdb yaml doc (apache#12460)
  [Modernizer] Add Maven Modernizer plugin in pulsar-proxy module (apache#12326)
  fix DefaultCryptoKeyReaderTest can not run on windows (apache#12475)
  apache#12429 only fixed the compactor skips data issue, but the normal reader/consumer (apache#12464)
  broker resource group test optimize fail msg (apache#12438)
  Stop OffsetStore when stopping the connector (apache#12457)
  fix a typo in UnAckedMessageTracker (apache#12467)
  docs(function): fix typo in pip install (apache#12468)
  Optimize the code: remove extra spaces (apache#12470)
  optimize SecurityUtility code flow (apache#12431)
  Update lombok to 1.18.22 (apache#12466)
  Update team.js to add David K. as a committer (apache#12440)
  Fix java demo error in reset cursor admin (apache#12454)
  ...

# Conflicts:
#	site2/website-next/versioned_docs/version-2.7.2/schema-evolution-compatibility.md
#	site2/website-next/versioned_docs/version-2.7.2/schema-get-started.md
#	site2/website-next/versioned_docs/version-2.7.2/schema-manage.md
#	site2/website-next/versioned_docs/version-2.7.2/schema-understand.md
#	site2/website-next/versioned_sidebars/version-2.7.2-sidebars.json
@urfreespace
Copy link
Member

LGTM @Anonymitaet

Yan Zhang and others added 10 commits October 26, 2021 15:01
* up/master:
  [C++] Fixed connection read error logging (apache#12492)
  [Pulsar SQL] Pulsar SQL support query big entry data (apache#12448)
  [Java Client] Remove data race in MultiTopicsConsumerImpl to ensure correct message order (apache#12456)
  Allow to have different instances LocalMemoryMetadataStore that share the same state (apache#12390)
  Remove unused ConsumerImpl.isTxnMessage (apache#12472)
* up/master:
  fix delete authentication policies when delete topic. (apache#12215)
  [Broker] Fix messageDedup delete inactive producer name (apache#12493)
  Fixed getting children of parent nodes in LocalMemoryMetadataStore (apache#12491)
  Update Producer stats on producer close() (apache#12500)
  docs(cli):add restart command in pulsar-daemon (apache#12373)
  Add the pulsar java property memory allocator doc (apache#12481)
  [Doc]Update ci-documentbot.yml (apache#12480)
* up/master:
  [pulsar-admin] Check validity of --compatibility for namespace (apache#12483)
  [website][upgrade]feat: version list page (apache#12511)
…ulsar into website/branch-2.7.2-chapter-4

* 'website/branch-2.7.2-chapter-4' of github.com:zeo1995/pulsar:
@Anonymitaet Anonymitaet merged commit 52dffda into apache:master Oct 28, 2021
eolivelli pushed a commit to eolivelli/pulsar that referenced this pull request Nov 29, 2021
…che#12394)

* doc migration version 2.7.2 chapter Pulsar Schema

* doc migration 2.7.2 chapter Pulsar IO

* Update the nesting of forms in the document

* update pulsar io

* patch

* commit 2.7.2 io quick start

* fix 2.7.3 and 2.8.0 and next

* fix develop tip

Co-authored-by: LiLi <urfreespace@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants