zachjsh commented on code in PR #15836:
URL: https://github.com/apache/druid/pull/15836#discussion_r1480474052
##########
sql/src/main/java/org/apache/druid/sql/calcite/parser/DruidSqlReplace.java:
##########
@@ -53,20 +55,41 @@ public class DruidSqlReplace extends DruidSqlIngest
*/
public DruidSqlReplace(
@Nonnull SqlInsert insertNode,
- @Nullable Granularity partitionedBy,
- @Nullable String partitionedByStringForUnparse,
+ @Nullable SqlNode partitionedBy,
@Nullable SqlNodeList clusteredBy,
@Nullable SqlNode replaceTimeQuery
)
{
- super(
+ this(
insertNode.getParserPosition(),
(SqlNodeList) insertNode.getOperandList().get(0), // No better getter
to extract this
Review Comment:
Seems that there is no getter for the fieldsList, and the fieldList is
hardcoded to be in this position in the operandList.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
