You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "jak78 (via GitHub)" <gi...@apache.org> on 2024/03/01 12:16:41 UTC

[PR] [fix][doc] Highlights that Pulsar clients can seamlessly access data in offloaded ledgers. [pulsar-site]

jak78 opened a new pull request, #810:
URL: https://github.com/apache/pulsar-site/pull/810

   This PR emphasizes that Pulsar clients can still transparently read the data in the offloaded ledgers.
   This should remove possible confusion, [as seen here](https://github.com/apache/pulsar/discussions/22113).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [fix][doc] Highlights that Pulsar clients can seamlessly access data in offloaded ledgers. [pulsar-site]

Posted by "visortelle (via GitHub)" <gi...@apache.org>.
visortelle commented on PR #810:
URL: https://github.com/apache/pulsar-site/pull/810#issuecomment-1974888186

   The reference to Pulsar SQL from tiered-storage-overview.md
   Looks good! 
   Merging.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [fix][doc] Highlights that Pulsar clients can seamlessly access data in offloaded ledgers. [pulsar-site]

Posted by "jak78 (via GitHub)" <gi...@apache.org>.
jak78 commented on code in PR #810:
URL: https://github.com/apache/pulsar-site/pull/810#discussion_r1509054392


##########
docs/tiered-storage-overview.md:
##########
@@ -83,6 +83,6 @@ Tiered storage works as follows:
 
 5. Moreover, you can trigger the offloading operation manually (via REST API or CLI) or automatically (via CLI).
 
-6. After offloading ledgers to long-term storage, you can still query data in the offloaded ledgers with Pulsar SQL.
+6. After transferring ledgers to long-term storage, the messages within these ledgers remain accessible to Pulsar consumers and readers, ensuring transparency in data retrieval. Additionally, you can query data in the offloaded ledgers using Pulsar SQL..

Review Comment:
   Good catch! I removed the sentence



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [fix][doc] Highlights that Pulsar clients can seamlessly access data in offloaded ledgers. [pulsar-site]

Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on code in PR #810:
URL: https://github.com/apache/pulsar-site/pull/810#discussion_r1508963937


##########
docs/tiered-storage-overview.md:
##########
@@ -83,6 +83,6 @@ Tiered storage works as follows:
 
 5. Moreover, you can trigger the offloading operation manually (via REST API or CLI) or automatically (via CLI).
 
-6. After offloading ledgers to long-term storage, you can still query data in the offloaded ledgers with Pulsar SQL.
+6. After transferring ledgers to long-term storage, the messages within these ledgers remain accessible to Pulsar consumers and readers, ensuring transparency in data retrieval. Additionally, you can query data in the offloaded ledgers using Pulsar SQL..

Review Comment:
   We have removed Pulsar SQL from the distribution since 3.2.0 . I wonder if this could be somehow communicated in this sentence or whether we should simply remove referencing Pulsar SQL.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [fix][doc] Highlights that Pulsar clients can seamlessly access data in offloaded ledgers. [pulsar-site]

Posted by "visortelle (via GitHub)" <gi...@apache.org>.
visortelle merged PR #810:
URL: https://github.com/apache/pulsar-site/pull/810


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [fix][doc] Highlights that Pulsar clients can seamlessly access data in offloaded ledgers. [pulsar-site]

Posted by "jak78 (via GitHub)" <gi...@apache.org>.
jak78 commented on PR #810:
URL: https://github.com/apache/pulsar-site/pull/810#issuecomment-1974898134

   Thanks!Le 2 mars 2024 à 20:34, Kiryl Valkovich ***@***.***> a écrit :
   Merged #810 into main.
   
   —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org