Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[FEATURE] Skip sending message to queue if number of SQL statements is 0 #2

Open
0xpetersatoshi opened this issue Apr 28, 2022 · 0 comments
Open

[FEATURE] Skip sending message to queue if number of SQL statements is 0 #2

0xpetersatoshi opened this issue Apr 28, 2022 · 0 comments
Assignees
@0xpetersatoshi
Labels
enhancement New feature or request

Comments

@0xpetersatoshi
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Currently, a message containing SQL statements is sent to an SQS queue so long as parsing the ABI does not fail. However, there is a large amount of ABIs that produce no SQL statements to be run. These should be filtered out and not sent to the queue for further processing as there is no further processing needed.

Describe the solution you'd like
Implement a step to filter these out and prevent them from being sent to the SQS queue.

Describe alternatives you've considered
N/A

Additional context
Should also investigate if getting 0 statements is valid to begin with. Does the contract not implement any methods/events or is this actually a bug?
@0xpetersatoshi 0xpetersatoshi added the enhancement New feature or request label Apr 28, 2022
@0xpetersatoshi 0xpetersatoshi self-assigned this Apr 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@0xpetersatoshi 0xpetersatoshi

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@0xpetersatoshi