Ankit Jain

Results 12 comments of Ankit Jain

Please also add the screenshot on how the output looks while running preview with the change.

> > Please also add the screenshot on how the output looks while running preview with the change. > > Added screenshot after fix ! Did we also run the...

Please resolve conflicts

Merged as part of https://github.com/data-integrations/google-cloud/pull/1001.

> > please add a test when both refresh_token & access_token are returned and only refresh_token was written. > > @itsankit-google @sahusanket I've been looking to add a test for...

I have the same issue, with `rootless: true` it does not have an option to accept user as input param: ``` 2025-06-11T15:08:15.3074031Z ##[group]Run docker/setup-docker-action@b60f85385d03ac8acfca6d9996982511d8620a19 2025-06-11T15:08:15.3074819Z with: 2025-06-11T15:08:15.3075003Z rootless: true 2025-06-11T15:08:15.3075380Z...

> > Can you please add the screenshot in the PR description on how the change was tested? > > Added a screenshot. What will be the case for existing...

> > What is the soft limit in apache POI by default? > > Apache poi uses `IOUtils.safelyAllocate` to allocate array of different size, the max len for each array...

> > What tests were performed to decide on this value? > > We wanted a value bigger than the current default, INT_MAX / 2 is taken as this is...