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

Figure out what to do with packages/@azure/eventhubs/testhub #1251

Closed
jeremymeng opened this issue Feb 16, 2019 · 5 comments · Fixed by #14678
Closed

Figure out what to do with packages/@azure/eventhubs/testhub #1251

jeremymeng opened this issue Feb 16, 2019 · 5 comments · Fixed by #14678
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. EngSys This issue is impacting the engineering system. Event Hubs
Milestone

Comments

@jeremymeng
Copy link
Member

This is a CLI testing tool to help test event hub sending and receiving. Should it belong to the SDK repository?

It also failed to build in the new repo when running npm install. One workaround could be referencing released NPM @azure/event-hubs and @azure/event-processor-host packages instead of pointing to client and processor source.

@jeremymeng jeremymeng added Client This issue points to a problem in the data-plane of the library. Event Hubs labels Feb 16, 2019
@AlexGhiondea
Copy link
Contributor

[triage] Let's move this into EPH folder.

@ramya-rao-a ramya-rao-a added this to the [2020] February milestone Dec 5, 2019
@ramya-rao-a
Copy link
Contributor

From #3956,

packages should follow the template structure and guidelines

testhub doesnt have a src folder

@chradek chradek modified the milestones: [2020] March, [2020] April Mar 9, 2020
@chradek chradek removed their assignment Mar 9, 2020
@chradek chradek modified the milestones: [2020] April, Backlog Apr 1, 2020
@ramya-rao-a ramya-rao-a added the EngSys This issue is impacting the engineering system. label May 22, 2020
@ramya-rao-a
Copy link
Contributor

This utility was used for testing Track 1 Event Hubs, but is no longer used. Before deleting it we should review it and understand what it was attempting to do and how. It can help us in the stress test framework we are building

@ramya-rao-a
Copy link
Contributor

Hey @HarshaNalluru

This package was put in place as a stress test for the older versions of the @azure/event-hubs package and the @azure/event-processor-host package. Now that we have a new framework for stress/perf tests, we should be able to remove this.

@richardpark-msft
Copy link
Member

CC: @chradek

@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. EngSys This issue is impacting the engineering system. Event Hubs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants