ResolvedNodes.ResolvedTableScan shouldn't require forSystemTimeExpr to be not null

[kanterov]

Generated code validates that forSystemTimeExpr isn't null:

protected void validate() {
    super.validate();
    Preconditions.checkArgument(this.table != null, "table must be set");
    Preconditions.checkArgument(this.forSystemTimeExpr != null, "forSystemTimeExpr must be set");
    Preconditions.checkArgument(this.alias != null, "alias must be set");
}

While NULL is a valid case if system time functionality isn't used.

[janhicken]

The validation here is generated and needs a general overhaul I think. For example, a having expression for an aggregation is also mandatory, or else the validation will throw an exception.

[serebro]

Hi guys. How to avoid this "forSystemTimeExpr must be set" exception. Which value I can use for forSystemTimeExpr as default to pass this checking?

[janhicken]

I actually tried to find a default value but failed to do so.

As a result, I created patches for this issue and also #31 and created my own build for our team to use 🙁

[matthewcbrown]

I think this should be fixed in the latest release.