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

feat: Remove deprecation (MAPCO-4117) #90

Merged
merged 21 commits into from
May 23, 2024
Merged

Conversation

razbroc
Copy link
Contributor

@razbroc razbroc commented Mar 18, 2024

Question Answer
Bug fix
New feature
Breaking change
Deprecations
Documentation
Tests added
Chore

Related issues: #MAPCO-4117

@razbroc razbroc requested a review from asafMasa March 18, 2024 16:11
@razbroc razbroc self-assigned this Mar 18, 2024
src/clients/jobManagerWrapper.ts Show resolved Hide resolved
src/common/interfaces.ts Show resolved Hide resolved
src/common/interfaces.ts Show resolved Hide resolved
src/common/interfaces.ts Show resolved Hide resolved
src/common/interfaces.ts Show resolved Hide resolved
src/common/interfaces.ts Show resolved Hide resolved
src/common/interfaces.ts Show resolved Hide resolved
src/common/interfaces.ts Show resolved Hide resolved
src/finalizationManager.ts Outdated Show resolved Hide resolved
@asafMasa asafMasa added the enhancement New feature or request label May 13, 2024
@asafMasa asafMasa changed the title chore: Remove deprecation (MAPCO-4117) feat: Remove deprecation (MAPCO-4117) May 13, 2024
@razbroc razbroc requested a review from asafMasa May 15, 2024 06:41
@razbroc razbroc merged commit 38f92ef into apply-tracing May 23, 2024
9 checks passed
@razbroc razbroc deleted the remove-deprecation branch May 23, 2024 15:50
razbroc added a commit that referenced this pull request Aug 1, 2024
* chore: add tracing config

* chore: apply tracing

* chore: configure ignored routes

* chore: add more tracing

* chore: modify gitignore

* fix: fix lint

* chore: remove trace from getTasksByJobId

* fix: pr fix

* fix: remove metrics buckets

* chore: upgrade telemetry version

* feat: Remove deprecation (MAPCO-4117) (#90)

* chore: remove create api

* chore: remove deprications

* chore: remove deprication

* chore: remove deprecated createMetadata

* chore: remove sendCallbacks

* fix: getInProgressJobs was using only get map jobs + removed deprications

* chore: remove ICallbackTarget

* chore: remove interface deprications

* chore: remove all deprecations for interfaces

* fix: lower coverage a tiny bit

* fix: fix lint

* fix: fix lint

* fix: fix lint

* fix: fix pr

* fix: fir pr comments

* chore: fix lint

* fix: fix pr

* fix: pr comment

* chore: upgrade telemetry pakcage

* chore: add parentContext to tasks traces

* chore: move fs to node:fs

* feat: update turf and commonjs to nodejs, span connection to parent

* fix: handle finalize implemantation

* chore: work on connected spans

* chore: tracing work in progress

* chore: finish up tracing pr

* chore:lint fix

* fix: fix lint

* fix:values tracing modification

* chore
:modify main span name

* fix: fix semantic pr comments

* chore: modify ItraceParentContext to SpanContext instead

* chore: update tracing configuration like boilerplate

* fix: fix pr comments

* fix: pr comments

* refactor: update debug log format

Co-authored-by: Shlomi k <[email protected]>

* refactor: remove not in use mock

* styple: prettier

---------

Co-authored-by: Shlomi k <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants