> 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 > >
