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.

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

Import is not working #355

Open
sarthakgarg814 opened this issue May 28, 2024 · 3 comments · May be fixed by #356
Open

Import is not working #355

sarthakgarg814 opened this issue May 28, 2024 · 3 comments · May be fixed by #356

Comments

@sarthakgarg814
Copy link
Contributor

Expected behavior

Actual behavior

TypeError: AsyncRetryHelper is not a constructor

Steps to reproduce the behavior

npm install and try

  1. Connect to Kafka cluster '...'
  2. Configure consumer with '...'
  3. Publish message '....'
  4. See error

Supporting materials

Additional Specs

  • Node.js version [e.g. 18.1.0]
  • Kafka version [e.g. 3.2.0]
  • KafkaJS version [e.g. 2.1.0]

Additional context
Add any other context about the problem here.

@brianphillips
Copy link
Collaborator

@sarthakgarg814 thanks for filing the issue. I'm assuming you are trying to use this module from within an ESM app? Would you mind trying kafkajs-async-retry@1.0.1-alpha.2 and seeing if it fixes the issue for you? I've changed how the NPM module is structured such that it includes both ESM and CJS transpiled versions.

@sarthakgarg814
Copy link
Contributor Author

Hi @brianphillips,

  1. Also one more issue is that we are not using producer.connect() in readme file.
  2. Auto Commit commits the message when we read it in retry queue but we sent it in pause state but when we resume the message is already acknowledged, hence we lost the message

Regards
Sarthak Garg

@brianphillips
Copy link
Collaborator

@sarthakgarg814 please file a new GitHub issue with more specific instructions (ideally, sample code) that demonstrates the issue(s) you're having. Also, please let me know once you've had a chance to try the version I mentioned and if it works in your environment, I'll release an actual v1.0.1 to fix the export/import issue.

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 a pull request may close this issue.

2 participants