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

v9.0.0 Ideas #791

Open
3 tasks
zemberdotnet opened this issue Sep 14, 2022 · 3 comments
Open
3 tasks

v9.0.0 Ideas #791

zemberdotnet opened this issue Sep 14, 2022 · 3 comments
Labels
breaking Will require a major version enhancement New feature or request proposal question Further information is requested

Comments

@zemberdotnet
Copy link
Member

zemberdotnet commented Sep 14, 2022 •
edited

Description

There are a number of changes, removals, and additions that we would like to do in a v9.0.0 of the SDK. This issue will serve as a collection of changes to consider in a v9.0.0 release:

  • Remove unhandledRejection listener
  • Remove multipleResolves listener
  • createIntegrationEntityV2

Submit any other v9.0.0 ideas below.
@zemberdotnet zemberdotnet added enhancement New feature or request question Further information is requested breaking Will require a major version proposal labels Sep 14, 2022
@zemberdotnet zemberdotnet pinned this issue Sep 14, 2022
@zemberdotnet
Copy link
Member Author

Disable createIntegrationEntity's automatic extraction of properties by default. (Keep flag or toggle to enable it)

@austinkelleher
Copy link
Contributor

@zemberdotnet

Disable createIntegrationEntity's automatic extraction of properties by default. (Keep flag or toggle to enable it)

Before we do this, we'd need to determine the impact of disabling it. It worries me that there are super large integrations (e.g. AWS, GCP, Azure) in which queries have been created that rely on the automatically extracted properties.

@zemberdotnet
Copy link
Member Author

zemberdotnet commented Oct 6, 2022 •
edited

@austinkelleher I've thought about this recently, and I now think that a createIntegrationEntityV2 may work better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking Will require a major version enhancement New feature or request proposal question Further information is requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@austinkelleher @zemberdotnet