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

[Task]: Make the 'log' object private in ActionBAse and BaseTransformMeta #4348

Closed

nadment opened this issue Sep 23, 2024 · 0 comments · Fixed by #4354

Assignees

Labels

P3 Pipelines task Transforms

Milestone

Contributor

nadment commented Sep 23, 2024 •

edited

Loading

What needs to happen?

When I see the fix to issue #4347, we should make the 'log' object private in the BaseTransformMeta class because it's not always initialized) and to force use the getLog() or logXXX methods.

Issue Priority

Priority: 3

Issue Component

Component: Pipelines

nadment added task Transforms labels

github-actions bot added P3 Pipelines labels

nadment self-assigned this

nadment added a commit to nadment/hop that referenced this issue


          Make the 'log' object private in the BaseTransformMeta apache#4348

dbd527d

nadment linked a pull request

that will close this issue

Make the 'log' object private in ActionBase and BaseTransformMeta #4348 #4354

Merged

nadment added a commit to nadment/hop that referenced this issue


          Make the 'log' object private in ActionBase and BaseTransformMeta apa…

aba1dde

…che#4348

nadment changed the title ~~[Task]: Make the 'log' object private in the BaseTransformMeta~~ [Task]: Make the 'log' object private in ActionBAse and BaseTransformMeta

nadment added a commit to nadment/hop that referenced this issue


          Make the 'log' object private in ActionBase and BaseTransformMeta apa…

f5a2e64

…che#4348

hansva closed this as completed in #4354

hansva added a commit that referenced this issue


          Merge pull request #4354 from nadment/4348

16d9d35

Make the 'log' object private in ActionBase and BaseTransformMeta #4348

hansva added this to the 2.11 milestone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment