Skip to content

Conversation

sgarg-CS
Copy link
Owner

PLUGIN-1137

This PR has been raised for internal review to be done by the CLOUDSUFI team members.

Please leave a comment here when you've reviewed all the changes in this PR.

Copy link
Collaborator

@kunalsharma-repo kunalsharma-repo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to me

@prashantcloudsufi
Copy link

@sgarg-CS Do we need this change as , if Servicenow is slow it is taking same time irrespective of the batch size as confirmed in the last call where poojan was running the postman

@sgarg-CS
Copy link
Owner Author

@prashantcloudsufi This feature doesn't make or break any existing functionality. It provides the user the flexibility to configure the number of records to be fetched in one batch from the UI at the design time itself or it can also be set as Macro and provided a value at the run time. It's an add-on that provides more control to the user running the pipeline.

In terms of improving the performance, we do not have the results at the moment as to how much more or less time the pipeline would take when run with lesser or greater page size. A guide on the Informatica website recommends having the batch size based on the data volume to improve the performance when data is read from ServiceNow. This feature will be useful here as the user can configure the size as per his requirement.

In case, this feature is not a part of this release, it could be considered as an enhancement for the future.

@prashantcloudsufi
Copy link

prashantcloudsufi commented Oct 11, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants