airavata-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Abhijit Karanjkar <abhijitkaranj...@gmail.com>
Subject Re: Thrift for User profile CPI
Date Tue, 08 Nov 2016 23:06:55 GMT
Thank you Supun,
I will look into it!

Regards,
Abhijit Karanjkar

On Tue, Nov 8, 2016 at 6:04 PM Supun Nakandala <supun.nakandala@gmail.com>
wrote:

> And also the build is failing in registry-refactoring branch.
>
> On Tue, Nov 8, 2016 at 6:00 PM, Supun Nakandala <supun.nakandala@gmail.com
> > wrote:
>
> Hi Abhijit,
>
> I merged your pull request.
>
> Few things,
>
> 1. I changed the name of the module from user-profile-cpi to
> user-profile-stubs
> 2. Please update the help section in the cpi thrift generation script to
> include the new command to generate user profile cpi.
> 3. Instead of RegistryServiceException please use a different exception
> (e.g UserProfileException) in the UserProfileService.
>
> On Tue, Nov 8, 2016 at 5:20 PM, Abhijit Karanjkar <
> abhijitkaranjkar@gmail.com> wrote:
>
> Hi Supun/Anuj,
> As discussed with Supun earlier, I have changed the location of user
> profile cpi files in "generate_cpi_stub.sh" so that the class files will be
> created
> in "../../modules/user-profile/user-profile-cpi/src/main/java"  whenever
> we run the .sh file mentioned above.
>
> Please pull the latest code from registry-refactoring branch and try
> running the file given above. Let me know if you face any issues.
>
> @Supun: Please correct me if I am wrong here.
>
> Regards,
> Abhijit Karanjkar
>
> On Fri, Nov 4, 2016 at 12:00 PM, Abhijit Karanjkar <
> abhijitkaranjkar@gmail.com> wrote:
>
> Hi Supun/Anuj
> I have added thrift file to generate User Profile CPI service under
> registry-refactoring branch.
> Could you please pull the changes and run generate_cpi_stub.sh? Let me
> know if it correctly generates the corresponding CPI files.
>
> Note: First Changes need to be merged to upstream, I have created a pull
> request.
>
> Thanks & regards,
> Abhijit Karanjkar
>
>
>
>
>
> --
> Thank you
> Supun Nakandala
> Dept. Computer Science and Engineering
> University of Moratuwa
>
>
>
>
> --
> Thank you
> Supun Nakandala
> Dept. Computer Science and Engineering
> University of Moratuwa
>

Mime
View raw message