Re: BEAM-9958: Code Review Wanted for PR 11674

2020-05-19 Thread Luke Cwik
Thanks Ahmet.

On Tue, May 19, 2020 at 1:26 PM Tomo Suzuki  wrote:

> Ahmet,
>
> Thank you for the review and merging.
>
> On Fri, May 15, 2020 at 2:06 PM Tomo Suzuki  wrote:
>
>> Hi Luke and Beam committers,
>>
>> Would you check this PR to use Linkage Checker's exclusion file?
>> https://github.com/apache/beam/pull/11674
>> This script used to use "diff" command to identify new linkage errors by
>> comparing line by line. With this PR, it identifies new linkage errors in
>> an appropriate manner. This PR opens up an opportunity to set up a Jenkins
>> job to check dependency conflicts.
>>
>> --
>> Regards,
>> Tomo
>>
>
>
> --
> Regards,
> Tomo
>


Re: BEAM-9958: Code Review Wanted for PR 11674

2020-05-19 Thread Tomo Suzuki
Ahmet,

Thank you for the review and merging.

On Fri, May 15, 2020 at 2:06 PM Tomo Suzuki  wrote:

> Hi Luke and Beam committers,
>
> Would you check this PR to use Linkage Checker's exclusion file?
> https://github.com/apache/beam/pull/11674
> This script used to use "diff" command to identify new linkage errors by
> comparing line by line. With this PR, it identifies new linkage errors in
> an appropriate manner. This PR opens up an opportunity to set up a Jenkins
> job to check dependency conflicts.
>
> --
> Regards,
> Tomo
>


-- 
Regards,
Tomo


BEAM-9958: Code Review Wanted for PR 11674

2020-05-15 Thread Tomo Suzuki
Hi Luke and Beam committers,

Would you check this PR to use Linkage Checker's exclusion file?
https://github.com/apache/beam/pull/11674
This script used to use "diff" command to identify new linkage errors by
comparing line by line. With this PR, it identifies new linkage errors in
an appropriate manner. This PR opens up an opportunity to set up a Jenkins
job to check dependency conflicts.

-- 
Regards,
Tomo