[DISCUSS]Review PR

[Andy <an...@163.com>]

Casion 09:38
https://github.com/apache/incubator-linkis/pull/2104 The pr that depends on the security upgrade of the component is free to help under review @legendtkl @peacewong@WDS 

legendtkl 09:40
receive

Casion 09:40
https://github.com/apache/incubator-linkis/pulls @Google Hua  @tikazhang  The pr of the new sqoop engine is available for review

peacewong@WDS 10:00
receive

peacewong@WDS 10:21
@legendtkl  Brother Kelu This spotless optimization PR also helps to review https://github.com/apache/incubator-linkis/pull/2107

legendtkl 11:12
done

peacewong@WDS 11:55
@jackxu2011_Linyun Xu Lingxiong There is conflict in this PR

Casion 12:01
@CCweixiao_intsig_leojie  https://github.com/apache/incubator-linkis/issues/2103 The JDBC engine reported by this user, the initialization call Kerberos authentication problem, please confirm it

Heisenberg 12:16
ok i'll take a look

Heisenberg 12:17
I will optimize this

Xu Ling 14:22
It should be caused by merging other pr first, because this pr will rearrange the import




casion 09:38
https://github.com/apache/incubator-linkis/pull/2104 依赖组件安全升级的pr 有空帮review下 @legendtkl @peacewong@WDS 

legendtkl 09:40
收到

casion 09:40
https://github.com/apache/incubator-linkis/pulls @Google Hua  @tikazhang  新增sqoop引擎的pr 有空帮review下

peacewong@WDS 10:00
收到

peacewong@WDS 10:21
@legendtkl 克路兄 这个spotless的优化PR也帮忙review下https://github.com/apache/incubator-linkis/pull/2107

legendtkl 11:12
done

peacewong@WDS 11:55
@jackxu2011_麟云 许灵兄 这个PR有冲突来着

casion 12:01
@CCweixiao_intsig_leojie  https://github.com/apache/incubator-linkis/issues/2103 这个用户反馈的 JDBC 引擎，初始化调用 Kerberos 身份验证问题，帮确认下 

海森堡 12:16
好的 我看下

海森堡 12:17
这个我优化一下

许灵 14:22
应该是先合并了其它的pr导致的，因为这个pr会重新排列import

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@linkis.apache.org
For additional commands, e-mail: dev-help@linkis.apache.org