wu-sheng commented on pull request #7444: URL: https://github.com/apache/skywalking/pull/7444#issuecomment-897299071
> > > > > @wu-sheng Pls kindly update the integration branch. Thx > > > > > > > > > > > > Why should we have this branch in this repo and sync it over and over? instead of creating it on @lujiajing1126 your fork repo? > > > > > > > > > The integration will not be merged to the master branch and will be pending util all features are ready (Trace, Log and Metrics). > > > > > > The codes can be in your fork repo always and you can sync from upstream at anytime you like, and do whatever you like until you think it's ready to open a pull request. > > Yes. What I was doing is fetching from upstream (i.e. Apache/Skywalking repo) and rebase from the `banyandb-integration` branch. So when I submit the PR, the compare view will be clear that it only shows what I have changed. Or it will be contaminated with codes from master. > > But I guess it would also be OK if you sync when I submit the next PR. Actually, before we merge the branch into the master, there is not necessary to keep the branch updated. The core of SkyWalking is quite stable, even in the 8.7.0 big perf improvement, there are not many codes to be changed. You focus on what you are doing, and resolve the conflicts, sync the codes when it is ready to run. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
