You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@metamodel.apache.org by Sagar Gandhi <ga...@live.com> on 2016/08/21 09:15:58 UTC
Re: Review Request 48005: Salesforce feature requests
> On May 28, 2016, 5:16 p.m., Kasper S�rensen wrote:
> > Thank you for the patch! It looks good to me code-wise. There's a mix of tabs and spaces which isn't consistent with the guidelines (but something that I could easily fix while applying the patch). For reference, we use (4) spaces instead of tabs for indentation.
Hi Kasper,
This code is not available in repo. Should I create issue in JIRA for this request ?. Please let me know if anything is required from my side.
Thanks,
Sagar Gandhi.
- Sagar
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48005/#review135389
-----------------------------------------------------------
On May 28, 2016, 8:50 a.m., Sagar Gandhi wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48005/
> -----------------------------------------------------------
>
> (Updated May 28, 2016, 8:50 a.m.)
>
>
> Review request for MetaModel.
>
>
> Repository: metamodel
>
>
> Description
> -------
>
> Feature Requests
>
> 1. Constructor which takes connection from calling program for hybrid Models.
> 2. Return the connection instance outside for hybrid model.
>
>
> Changes done
> 1. Added constructor which takes PartnerConnection as argument.
> 2. Added getter method for ParterConnection
>
> Open Questions
> 1. Currently I have validated the input PartenerConnection for null value. Do we also need to check if connection is alive or not ? . If given input connection is already logged out then the further operations may fail with exception. There is a similar situation in JDBCDataContext which takes Connection as in parameter, but in JDBCContext we are performing operation on input connection in constructor only. So if connection is already closed then constructor only logs the exception to warn or debug level. It does not throw exception.
>
>
> Diffs
> -----
>
> salesforce/src/main/java/org/apache/metamodel/salesforce/SalesforceDataContext.java c8add52
>
> Diff: https://reviews.apache.org/r/48005/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Sagar Gandhi
>
>