Re: Build issues from SENTRY-2208

2018-05-31 Thread Sergio Pena
Yes, all contributors have access to the apache repo. https://git-wip-us.apache.org/repos/asf/sentry.git On Thu, May 31, 2018 at 10:30 AM, Na Li wrote: > Sergio, > > Do all contributors have access to get code from apache repo for > committers? > > If not, then the sync issue still affects those

Re: Build issues from SENTRY-2208

2018-05-31 Thread Na Li
Sergio, Do all contributors have access to get code from apache repo for committers? If not, then the sync issue still affects those contributors. Lina On Thu, May 31, 2018 at 9:51 AM, Sergio Pena wrote: > I reverted and reapplied the patch last night with the smart-apply-patch.sh > script wh

Re: Build issues from SENTRY-2208

2018-05-31 Thread Sergio Pena
I reverted and reapplied the patch last night with the smart-apply-patch.sh script which was used for running the tests. Is it still failing? Btw, I don't think we should get blocked by the out of sync issues between github and the apache repo. If we use the apache repo for commits and development

Re: Build issues from SENTRY-2208

2018-05-30 Thread Stephen Moist
I would see what’s causing the delay of synching the repos. But as long as there are no other changes that affect SENTRY-2208, I’d be fine with that. > On May 30, 2018, at 4:19 PM, Kalyan Kumar Kalvagadda > wrote: > > Steve, > > This is blocking other commits. Can we revert SENTRY-2208 ( > b

Re: Build issues from SENTRY-2208

2018-05-30 Thread Kalyan Kumar Kalvagadda
Steve, This is blocking other commits. Can we revert SENTRY-2208 ( b97f5c7aa89d3c9061dd1b6655fc0702caa93f1f) but hold on committing again till Friday. Huge patches like this are making github and apache repositories out of sync for a couple of days, blocking every one submitting patches. *Thank