怎样向社区贡献代码

最近向社区提交了一个RP。发现RP一直在等待管理员批准进入测试等环节,没人批准也没人反馈。想问一下怎样才能快速、顺利的向StarRocks社区做贡献?

你的 PR 可以来个 link 吗?或是你的 GitHub ID 来一下,我看看是什么问题~论坛这个昵称木有搜到 :cry:

:+1: :+1:昨晚有进展了,检查出了一个问题,刚刚修复了现在git在做检查。这个是我的RP的地址:https://github.com/StarRocks/starrocks/pull/47862。

看起来现在又不动了。@Kate_Shao 麻烦帮忙看一下是什么回事。RP的连接是: https://github.com/StarRocks/starrocks/pull/47862

已经在跑测试了,前面 PR 有点多所以等待一些时间都算是正常的情况~

:+1: :+1:谢谢!
但现在发现下面这个错误,这个跟好像我的RP没关,是别的模块的测试:
*** SIGABRT (@0x73aa0) received by PID 473760 (TID 0x7f85f3b1f180) from PID 473760; stack trace: ***

8023 @ 0x12ba58a2 google::(anonymous namespace)::FailureSignalHandler()

8024 @ 0x7f85f2145630 (unknown)

8025 @ 0x7f85f1260387 __GI_raise

8026 @ 0x7f85f1261a78 __GI_abort

8027 @ 0xa25cd70 starrocks::failure_function()

8028 @ 0x12b929ad google::LogMessage::Fail()

8029 @ 0x12b94e1f google::LogMessage::SendToLog()

8030 @ 0x12b924fe google::LogMessage::Flush()

8031 @ 0x12b95429 google::LogMessageFatal::~LogMessageFatal()

8032 @ 0x8e78770 starrocks::lake::LakeScanNodeTest::create_rowsets_for_testing()

8033 @ 0x8e73b72 starrocks::lake::LakeScanNodeTest_test_issue_44386_Test::TestBody()

8034 @ 0x14df5251 testing::internal::HandleExceptionsInMethodIfSupported<>()

8035 @ 0x14de7226 testing::Test::Run()

8036 @ 0x14de7395 testing::TestInfo::Run()

8037 @ 0x14de7485 testing::TestSuite::Run()

8038 @ 0x14de79d6 testing::internal::UnitTestImpl::RunAllTests()

8039 @ 0x14de7bde testing::UnitTest::Run()

8040 @ 0x6b97f31 starrocks::init_test_env()

8041 @ 0x6b9eae2 main

8042 @ 0x7f85f124c555 __libc_start_main

8043 @ 0x6b94029 (unknown)

8044 @ 0x0 (unknown)

我这边也有一个提交,都好几个月了,还卡在最后一步。。

我帮你去摇人 review 代码~

@Kate_Shao我的RP有个spl 测试不通过,修复了重新提交代码了。 麻烦再帮我触发一次流程。

似乎还是少人审核。。。

@Kate_Shao 很抱歉!由于我对sql的测试规范不是很了解,有个sql的预期测试结果写错了,导致这个sql一直测试不通过。刚才在本地运行了python3 run.py -d ./sql/test_array -c 1 -v,找到了问题所在并进行了修复在本地进行sql测试也通过了。麻烦再帮我摇人 review 一下。

@Kate_Shao 小仙女,帮帮忙呀!
能不能帮我看看为什么我的RP的codecov/project/be-total里有很多代码路径都没有走到;但我在本地进行调试时看到单元测试时,我实现的方法的所有代码路径都走到了。或者能不能帮我找人看看是什么回事?

CI check里, 没有标记为required的检查, 可以不用管. be-total目前没有强制要求, 只是一个参考指标.

ok,谢谢!

@Kate_Shao 可不可以再帮我通知一下kangkaisen 把我的代码review了,让我的RP尽快合并了。我担心我建新分支会触发分叉同步,又要重新走流程了。

@Kate_Shao :+1: :+1: 终于合并了,感谢!

1赞

客气客气,也感谢你帮社区添砖加瓦。来填个表单拿奖品咯~ :point_down: