Re: [I] [Bug]: Excel Input eats incoming fields, but still passes the metadata for them downstream (as if they still exist) (hop)

2023-12-27 Thread via GitHub
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

Re: [I] [Bug]: Excel Input eats incoming fields, but still passes the metadata for them downstream (as if they still exist) (hop)

2023-11-22 Thread via GitHub
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

Re: [I] [Bug]: Excel Input eats incoming fields, but still passes the metadata for them downstream (as if they still exist) (hop)

2023-11-22 Thread via GitHub
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

Re: [I] [Bug]: Excel Input eats incoming fields, but still passes the metadata for them downstream (as if they still exist) (hop)

2023-11-21 Thread via GitHub
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.

Re: [I] [Bug]: Excel Input eats incoming fields, but still passes the metadata for them downstream (as if they still exist) (hop)

2023-11-20 Thread via GitHub
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

Re: [I] [Bug]: Excel Input eats incoming fields, but still passes the metadata for them downstream (as if they still exist) (hop)

2023-11-20 Thread via GitHub
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

Re: [I] [Bug]: Excel Input eats incoming fields, but still passes the metadata for them downstream (as if they still exist) (hop)

2023-11-18 Thread via GitHub
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

Re: [I] [Bug]: Excel Input eats incoming fields, but still passes the metadata for them downstream (as if they still exist) (hop)

2023-11-18 Thread via GitHub
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)