vbhanuchander-lang commented on PR #7859: URL: https://github.com/apache/hop/pull/7859#issuecomment-5247093171
Understood, thanks for the heads-up @hansva — no problem waiting for 2.20, and I appreciate you milestoning it rather than leaving it in limbo. It is ready whenever you pick it up: the javadoc failure that made the Code job red is fixed in 802b850, and I have re-verified `javadoc:javadoc` and the module's Spark SQL tests (18) since. Nothing outstanding from my side. Two open questions from the description are still worth your view when you get to it, since both would change the shape of the transform rather than being add-ons: 1. A **Get fields** button that runs the statement to populate the output field list — this needs a decision on where a design-time `SparkSession` comes from, which is why I left it out of v1. 2. Whether the required explicit output field list is the right contract, or whether you would prefer schema inference with the field list optional. I will not push anything further to this branch unless you ask, so it stays stable for your review after the release. -- 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 specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
