Hi, I have made the suggested changes[1] except for not using a tempKey in addPair() method in SimpleObjectUnionConstructor: tempKey is needed here because a UTF8StringPointable is needed to be passed to the getValue method of the ObjectPointable class.
[1] https://github.com/apache/vxquery/pull/99 Thank you. Yours sincerely, Riyafa On 15 July 2016 at 06:08, Riyafa Abdul Hameed <[email protected]> wrote: > Hi, > > I have made a PR with suggested improvements to > SimpleObjectUnionScalarEvaluator so that it doesn't extend the > ObjectConstructorScalarEvaluator[1]. > > [1] https://github.com/apache/vxquery/pull/99 > > Thank you. > > Yours sincerely, > Riyafa > > -- > Riyafa Abdul Hameed > Undergraduate, University of Moratuwa > > Email: [email protected] > Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/> > <http://facebook.com/riyafa.ahf> <http://lk.linkedin.com/in/riyafa> > <http://twitter.com/Riyafa1> > -- Riyafa Abdul Hameed Undergraduate, University of Moratuwa Email: [email protected] Website: https://riyafa.wordpress.com/ <http://riyafa.wordpress.com/> <http://facebook.com/riyafa.ahf> <http://lk.linkedin.com/in/riyafa> <http://twitter.com/Riyafa1>
