Re: StandaloneWalRecordsIterator: support iteration from custom pointer

2018-08-17 Thread Ivan Rakov

Dmitriy,

Thanks a lot for stepping in! Your help is appreciated.
I've looked through your changes and left a comment in JIRA issue.

Best Regards,
Ivan Rakov

On 16.08.2018 23:56, Dmitriy Govorukhin wrote:

Ivan,

I implemented this issue, please review my changes.
https://reviews.ignite.apache.org/ignite/review/IGNT-CR-729

On Thu, Aug 16, 2018 at 3:09 PM Ivan Rakov  wrote:


Thanks for your comments!
I've created a ticket: https://issues.apache.org/jira/browse/IGNITE-9294

Best Regards,
Ivan Rakov

On 15.08.2018 21:31, Dmitriy Setrakyan wrote:

Agree, this should be a great performance boost.

On Wed, Aug 15, 2018 at 10:17 AM, Dmitriy Pavlov 
wrote:


Hi Ivan,

I agree that providing WAL pointer is the better option. Initially,
Standalone WAL iterator was developed for debugging utility, so a set of
files was perfectly OK.

Sincerely,
Dmitriy Palov

ср, 15 авг. 2018 г. в 20:06, Ivan Rakov :


Igniters,

Right now we are developing WAL shipping process for our internal
purposes and we use StandaloneWalRecordsIterator to read WAL files from
custom destination. We have bumped into a problem - iterator can be
constructed from set of files and dirs, but there's no option to pass
WAL pointer to the iterator factory class to start iteration with. It
can be worked around (by filtering all records prior to needed

pointer),

but I think it would be handy to add such option to
IgniteWalIteratorFactory API.

What do you think?

--
Best Regards,
Ivan Rakov








Re: StandaloneWalRecordsIterator: support iteration from custom pointer

2018-08-16 Thread Dmitriy Govorukhin
Ivan,

I implemented this issue, please review my changes.
https://reviews.ignite.apache.org/ignite/review/IGNT-CR-729

On Thu, Aug 16, 2018 at 3:09 PM Ivan Rakov  wrote:

> Thanks for your comments!
> I've created a ticket: https://issues.apache.org/jira/browse/IGNITE-9294
>
> Best Regards,
> Ivan Rakov
>
> On 15.08.2018 21:31, Dmitriy Setrakyan wrote:
> > Agree, this should be a great performance boost.
> >
> > On Wed, Aug 15, 2018 at 10:17 AM, Dmitriy Pavlov 
> > wrote:
> >
> >> Hi Ivan,
> >>
> >> I agree that providing WAL pointer is the better option. Initially,
> >> Standalone WAL iterator was developed for debugging utility, so a set of
> >> files was perfectly OK.
> >>
> >> Sincerely,
> >> Dmitriy Palov
> >>
> >> ср, 15 авг. 2018 г. в 20:06, Ivan Rakov :
> >>
> >>> Igniters,
> >>>
> >>> Right now we are developing WAL shipping process for our internal
> >>> purposes and we use StandaloneWalRecordsIterator to read WAL files from
> >>> custom destination. We have bumped into a problem - iterator can be
> >>> constructed from set of files and dirs, but there's no option to pass
> >>> WAL pointer to the iterator factory class to start iteration with. It
> >>> can be worked around (by filtering all records prior to needed
> pointer),
> >>> but I think it would be handy to add such option to
> >>> IgniteWalIteratorFactory API.
> >>>
> >>> What do you think?
> >>>
> >>> --
> >>> Best Regards,
> >>> Ivan Rakov
> >>>
> >>>
>
>


Re: StandaloneWalRecordsIterator: support iteration from custom pointer

2018-08-16 Thread Ivan Rakov

Thanks for your comments!
I've created a ticket: https://issues.apache.org/jira/browse/IGNITE-9294

Best Regards,
Ivan Rakov

On 15.08.2018 21:31, Dmitriy Setrakyan wrote:

Agree, this should be a great performance boost.

On Wed, Aug 15, 2018 at 10:17 AM, Dmitriy Pavlov 
wrote:


Hi Ivan,

I agree that providing WAL pointer is the better option. Initially,
Standalone WAL iterator was developed for debugging utility, so a set of
files was perfectly OK.

Sincerely,
Dmitriy Palov

ср, 15 авг. 2018 г. в 20:06, Ivan Rakov :


Igniters,

Right now we are developing WAL shipping process for our internal
purposes and we use StandaloneWalRecordsIterator to read WAL files from
custom destination. We have bumped into a problem - iterator can be
constructed from set of files and dirs, but there's no option to pass
WAL pointer to the iterator factory class to start iteration with. It
can be worked around (by filtering all records prior to needed pointer),
but I think it would be handy to add such option to
IgniteWalIteratorFactory API.

What do you think?

--
Best Regards,
Ivan Rakov






[jira] [Created] (IGNITE-9294) StandaloneWalRecordsIterator: support iteration from custom pointer

2018-08-16 Thread Ivan Rakov (JIRA)
Ivan Rakov created IGNITE-9294:
--

 Summary: StandaloneWalRecordsIterator: support iteration from 
custom pointer
 Key: IGNITE-9294
 URL: https://issues.apache.org/jira/browse/IGNITE-9294
 Project: Ignite
  Issue Type: Improvement
  Components: persistence
Reporter: Ivan Rakov


StandaloneWalRecordsIterator can be constructed from set of files and dirs, but 
there's no option to pass WAL pointer to the iterator factory class to start 
iteration with. It can be worked around (by filtering all records prior to 
needed pointer), but also it would be handy to add such option to 
IgniteWalIteratorFactory API.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


Re: StandaloneWalRecordsIterator: support iteration from custom pointer

2018-08-15 Thread Dmitriy Setrakyan
Agree, this should be a great performance boost.

On Wed, Aug 15, 2018 at 10:17 AM, Dmitriy Pavlov 
wrote:

> Hi Ivan,
>
> I agree that providing WAL pointer is the better option. Initially,
> Standalone WAL iterator was developed for debugging utility, so a set of
> files was perfectly OK.
>
> Sincerely,
> Dmitriy Palov
>
> ср, 15 авг. 2018 г. в 20:06, Ivan Rakov :
>
> > Igniters,
> >
> > Right now we are developing WAL shipping process for our internal
> > purposes and we use StandaloneWalRecordsIterator to read WAL files from
> > custom destination. We have bumped into a problem - iterator can be
> > constructed from set of files and dirs, but there's no option to pass
> > WAL pointer to the iterator factory class to start iteration with. It
> > can be worked around (by filtering all records prior to needed pointer),
> > but I think it would be handy to add such option to
> > IgniteWalIteratorFactory API.
> >
> > What do you think?
> >
> > --
> > Best Regards,
> > Ivan Rakov
> >
> >
>


Re: StandaloneWalRecordsIterator: support iteration from custom pointer

2018-08-15 Thread Dmitriy Pavlov
Hi Ivan,

I agree that providing WAL pointer is the better option. Initially,
Standalone WAL iterator was developed for debugging utility, so a set of
files was perfectly OK.

Sincerely,
Dmitriy Palov

ср, 15 авг. 2018 г. в 20:06, Ivan Rakov :

> Igniters,
>
> Right now we are developing WAL shipping process for our internal
> purposes and we use StandaloneWalRecordsIterator to read WAL files from
> custom destination. We have bumped into a problem - iterator can be
> constructed from set of files and dirs, but there's no option to pass
> WAL pointer to the iterator factory class to start iteration with. It
> can be worked around (by filtering all records prior to needed pointer),
> but I think it would be handy to add such option to
> IgniteWalIteratorFactory API.
>
> What do you think?
>
> --
> Best Regards,
> Ivan Rakov
>
>


StandaloneWalRecordsIterator: support iteration from custom pointer

2018-08-15 Thread Ivan Rakov

Igniters,

Right now we are developing WAL shipping process for our internal 
purposes and we use StandaloneWalRecordsIterator to read WAL files from 
custom destination. We have bumped into a problem - iterator can be 
constructed from set of files and dirs, but there's no option to pass 
WAL pointer to the iterator factory class to start iteration with. It 
can be worked around (by filtering all records prior to needed pointer), 
but I think it would be handy to add such option to 
IgniteWalIteratorFactory API.


What do you think?

--
Best Regards,
Ivan Rakov