xuzifu666
xuzifu666
@bchapuis have a review please, thanks
> Sorry, I'm not a commiter to Apache Calcite and lack context to form an opinion on this change. get
@danny0405 Teacher Chan, the api is not used in Calcite, want to refactor it throght remove it for other developers not use the api,thanks
7958 or 7954 @Johnsonginati
java ci error seem not relate to the changed code,and azure is success
> @xuzifu666 Thanks for contribution. This pr add throw exception in each write handlers. Is it possible to check all write statuses for failures during the commit phase? Good advice,I...
CI error seems not relete to the changed code,and Azure CI is success @danny0405
@bvaradar All had addressed,could you give the final review and give more advices?
> Do you think you can provide a test coverage to protect your contribution from potential future regression, @xuzifu666 ? > > > Not need @dongjoon-hyun Thanks for your attentions,In...
@dongjoon-hyun Could you give a final review? Thanks