Hi, I misread the documentation in the patch. Thank you for pointing out my mistake. Best Regards Ian
On 6 March 2018 at 09:53, Tomek Rękawek <[email protected]> wrote: > Hi Ian, > > > On 5 Mar 2018, at 17:47, Ian Boston <[email protected]> wrote: > > > > I assume that the patch deals with the 50K limit[1] to the number of > blocks > > per Azure Blob store ? > > As far as I understand, it’s the limit that applies to the number of > blocks in a single blob. Block is a single write. Since the segments are > immutable (written at once), we don’t need to worry about this limit for > the segments. It’s a different case for the journal file - a single commit > leads to a single append which adds a block. However, the patch takes care > of this, by creating journal.log.001, .002, when we’re close to the limit > [1]. > > Regards, > Tomek > > [1] https://github.com/trekawek/jackrabbit-oak/blob/OAK-6922/ > oak-segment-azure/src/main/java/org/apache/jackrabbit/oak/segment/azure/ > AzureJournalFile.java#L37 > > -- > Tomek Rękawek | Adobe Research | www.adobe.com > [email protected] > >
