You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@beam.apache.org by Pulasthi Supun Wickramasinghe <pu...@gmail.com> on 2020/09/01 02:49:20 UTC
Adding Twister2 documentation
Hi Devs.
I created two PR's to add the missing Twister2 runner documentation[1] [2],
however, would need some guidance on how to fill out the compatibility
matrix.
1. Is there some mapping between test classes and entries in the
compatibility matrix I can use to crosscheck and tick off boxes in the
matrix.
2. Since the Twister2 runner only supports batch mode at the moment how
would I reflect that in the compatibility matrix?
[1] https://github.com/apache/beam/pull/12731
[2] https://github.com/apache/beam/pull/12744
Best Regards,
Pulasthi
--
Pulasthi S. Wickramasinghe
PhD Candidate | Research Assistant
School of Informatics and Computing | Digital Science Center
Indiana University, Bloomington
cell: 224-386-9035
Re: Adding Twister2 documentation
Posted by Pulasthi Supun Wickramasinghe <pu...@gmail.com>.
Hi Ismaël
Thank you for the review and merge. We hope to keep improving the Twister2
runner as the frameworks develops as well. I will also try to help out more
within the community since i gained a bit of experience with the runners
developing this one.
Best Regards,
Pulasthi
On Tue, Sep 15, 2020 at 12:06 PM Ismaël Mejía <ie...@gmail.com> wrote:
> Hi, both PRs were reviewed/merged yesterday, thanks for adding more docs
> and
> keep improving the runner!
>
> I would also like to ask the other members of the community for help in
> general
> in circumstances like this one. One argument to not help with these
> reviews is
> the lack of knowledge of the target system (Twister2 in this case) but
> anyone
> familiar with MapReduce like systems and Beam can definitely give a hand
> here
> to avoid making me or others a SPOF for reviews.
>
> On Thu, Sep 10, 2020 at 7:44 PM Pulasthi Supun Wickramasinghe
> <pu...@gmail.com> wrote:
> >
> > @Ismaël Mejía
> >
> > would you have some cycles to validates these changes?
> >
> > Best Regards,
> > Pulasthi
> >
> > On Mon, Aug 31, 2020 at 10:49 PM Pulasthi Supun Wickramasinghe <
> pulasthi911@gmail.com> wrote:
> >>
> >> Hi Devs.
> >>
> >> I created two PR's to add the missing Twister2 runner documentation[1]
> [2], however, would need some guidance on how to fill out the compatibility
> matrix.
> >>
> >> Is there some mapping between test classes and entries in the
> compatibility matrix I can use to crosscheck and tick off boxes in the
> matrix.
> >> Since the Twister2 runner only supports batch mode at the moment how
> would I reflect that in the compatibility matrix?
> >>
> >> [1] https://github.com/apache/beam/pull/12731
> >> [2] https://github.com/apache/beam/pull/12744
> >>
> >> Best Regards,
> >> Pulasthi
> >> --
> >> Pulasthi S. Wickramasinghe
> >> PhD Candidate | Research Assistant
> >> School of Informatics and Computing | Digital Science Center
> >> Indiana University, Bloomington
> >> cell: 224-386-9035
> >
> >
> >
> > --
> > Pulasthi S. Wickramasinghe
> > PhD Candidate | Research Assistant
> > School of Informatics and Computing | Digital Science Center
> > Indiana University, Bloomington
> > cell: 224-386-9035
>
--
Pulasthi S. Wickramasinghe
PhD Candidate | Research Assistant
School of Informatics and Computing | Digital Science Center
Indiana University, Bloomington
cell: 224-386-9035
Re: Adding Twister2 documentation
Posted by Ismaël Mejía <ie...@gmail.com>.
Hi, both PRs were reviewed/merged yesterday, thanks for adding more docs and
keep improving the runner!
I would also like to ask the other members of the community for help in general
in circumstances like this one. One argument to not help with these reviews is
the lack of knowledge of the target system (Twister2 in this case) but anyone
familiar with MapReduce like systems and Beam can definitely give a hand here
to avoid making me or others a SPOF for reviews.
On Thu, Sep 10, 2020 at 7:44 PM Pulasthi Supun Wickramasinghe
<pu...@gmail.com> wrote:
>
> @Ismaël Mejía
>
> would you have some cycles to validates these changes?
>
> Best Regards,
> Pulasthi
>
> On Mon, Aug 31, 2020 at 10:49 PM Pulasthi Supun Wickramasinghe <pu...@gmail.com> wrote:
>>
>> Hi Devs.
>>
>> I created two PR's to add the missing Twister2 runner documentation[1] [2], however, would need some guidance on how to fill out the compatibility matrix.
>>
>> Is there some mapping between test classes and entries in the compatibility matrix I can use to crosscheck and tick off boxes in the matrix.
>> Since the Twister2 runner only supports batch mode at the moment how would I reflect that in the compatibility matrix?
>>
>> [1] https://github.com/apache/beam/pull/12731
>> [2] https://github.com/apache/beam/pull/12744
>>
>> Best Regards,
>> Pulasthi
>> --
>> Pulasthi S. Wickramasinghe
>> PhD Candidate | Research Assistant
>> School of Informatics and Computing | Digital Science Center
>> Indiana University, Bloomington
>> cell: 224-386-9035
>
>
>
> --
> Pulasthi S. Wickramasinghe
> PhD Candidate | Research Assistant
> School of Informatics and Computing | Digital Science Center
> Indiana University, Bloomington
> cell: 224-386-9035
Re: Adding Twister2 documentation
Posted by Pulasthi Supun Wickramasinghe <pu...@gmail.com>.
@Ismaël Mejía <ie...@gmail.com>
would you have some cycles to validates these changes?
Best Regards,
Pulasthi
On Mon, Aug 31, 2020 at 10:49 PM Pulasthi Supun Wickramasinghe <
pulasthi911@gmail.com> wrote:
> Hi Devs.
>
> I created two PR's to add the missing Twister2 runner documentation[1]
> [2], however, would need some guidance on how to fill out the compatibility
> matrix.
>
>
> 1. Is there some mapping between test classes and entries in the
> compatibility matrix I can use to crosscheck and tick off boxes in the
> matrix.
> 2. Since the Twister2 runner only supports batch mode at the moment
> how would I reflect that in the compatibility matrix?
>
> [1] https://github.com/apache/beam/pull/12731
> [2] https://github.com/apache/beam/pull/12744
>
> Best Regards,
> Pulasthi
> --
> Pulasthi S. Wickramasinghe
> PhD Candidate | Research Assistant
> School of Informatics and Computing | Digital Science Center
> Indiana University, Bloomington
> cell: 224-386-9035
>
--
Pulasthi S. Wickramasinghe
PhD Candidate | Research Assistant
School of Informatics and Computing | Digital Science Center
Indiana University, Bloomington
cell: 224-386-9035