usbrandon closed issue #3221: [Bug]: Excel Input eats incoming fields, but
still passes the metadata for them downstream (as if they still exist)
URL: https://github.com/apache/hop/issues/3221
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
usbrandon commented on issue #3221:
URL: https://github.com/apache/hop/issues/3221#issuecomment-1823590974
Hi! I am delighted to have a fix. I will create a separate feature
request for the other. The main issue you solved was actually making a big
problem, the feature is just a
sramazzina commented on issue #3221:
URL: https://github.com/apache/hop/issues/3221#issuecomment-1823572684
Hi @usbrandon, after a careful evaluation I decided to proceed differently
and just fix the issue that let unwanted metadata move along.
About the idea of having a flag that
sramazzina commented on issue #3221:
URL: https://github.com/apache/hop/issues/3221#issuecomment-1820440862
Ciao @usbrandon thanks for your answer. The fix is almost done I was just
waiting your confirmation to complete it. I will let you know when it will be
ready so that you can test it.
usbrandon commented on issue #3221:
URL: https://github.com/apache/hop/issues/3221#issuecomment-1819764434
Hello @sramazzina! Thank you for taking a look. I agree with you on both
fronts. By default the transform should not emit the incoming fields or
metadata. If there were a checkbox
sramazzina commented on issue #3221:
URL: https://github.com/apache/hop/issues/3221#issuecomment-1819255668
Ciao @usbrandon. The standard behavior of this transform is to stop the
incoming flow to go through the Excel Input. Overall, the best option in my
opinion is to fix the discrepancy
bjackson-ep commented on issue #3221:
URL: https://github.com/apache/hop/issues/3221#issuecomment-1817724853
My included example is 2.6-GA
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the
bjackson-ep commented on issue #3221:
URL: https://github.com/apache/hop/issues/3221#issuecomment-1817724724
[excel-different-layout-solution.zip](https://github.com/apache/hop/files/13401961/excel-different-layout-solution.zip)