damavis / airflow-pentaho-plugin

Pentaho plugin for Apache Airflow - Orquestate pentaho transformations and jobs from Airflow
Apache License 2.0
39 stars 17 forks source link

cannot get transformation result status Carte #34

Closed 3pm closed 11 months ago

3pm commented 11 months ago

Hello! Pentaho 9.4 + Carte When using CarteTransOperator it cannot get execution status: airflow.exceptions.AirflowException: Unexpected server response: {"webresult": {"result": "ERROR", "message": "The specified transformation [trans.ktr] could not be found", "id": null}}

It's because it passes .ktr along to transformation name. If ".ktr" is removed in _get_trans_name (carte.py file) - it works.

That's not the case in CarteJobOperator, it works.

piffall commented 11 months ago

Hi @3pm, did you try to use just the name, without the extension? I think that the .ktr is not necessary.

3pm commented 11 months ago

Hello. The only place I pass path is trans='/home/xxx/pentaho/local/tmp/trans.ktr' parameter. Without full path (including .ktr) it is not working. Later the name is taken from this path in CarteTransOperator.

3pm commented 11 months ago

This helps for me: def _get_trans_name(self): return self.trans.split('/').pop().replace(".ktr","")

piffall commented 11 months ago

Fixed in v1.1.2: https://github.com/damavis/airflow-pentaho-plugin/commit/87895d1b469e0180be1e380a26d70d4c420d75f9

3pm commented 5 months ago

Hello, can you put it on release 1.1.2 please?

piffall commented 5 months ago

That should be already there @3pm

3pm commented 5 months ago

Hello, I'm sorry. Tank you. Looked at the wrong place.

On Tue, 7 May 2024, 16:59 Cristòfol Torrens, @.***> wrote:

That should be already there @3pm https://github.com/3pm

— Reply to this email directly, view it on GitHub https://github.com/damavis/airflow-pentaho-plugin/issues/34#issuecomment-2098477469, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAO5PKGHE5ENO7CX7BEZQUTZBDM2RAVCNFSM6AAAAAA63CUKTSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJYGQ3TONBWHE . You are receiving this because you were mentioned.Message ID: @.***>