Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Aggregate DateTime columns are realized without TZs #429

Closed
ldhasson opened this issue Sep 13, 2019 · 0 comments
Closed

Aggregate DateTime columns are realized without TZs #429

ldhasson opened this issue Sep 13, 2019 · 0 comments
Labels
bug S1

Comments

@ldhasson
Copy link
Collaborator

ldhasson commented Sep 13, 2019 •
edited
Loading

_a9TZ cannot be resolved to a variable TILDA__TESTINGVIEW.java
_testFirstTZ cannot be resolved to a variable TILDA__TEST_REALIZED.java
_testFirstTZ cannot be resolved to a variable TILDA__TEST_VIEW.java
_testLastTZ cannot be resolved to a variable TILDA__TEST_REALIZED.java
_testLastTZ cannot be resolved to a variable TILDA__TEST_VIEW.java
A9TZ cannot be resolved or is not a field TILDA__TESTINGVIEW.java
TESTFIRSTTZ cannot be resolved or is not a field TILDA__TEST_REALIZED.java
TESTLASTTZ cannot be resolved or is not a field TILDA__TEST_REALIZED.java
The method setNullTestFirstTZ() is undefined for the type TILDA__TEST_REALIZED TILDA__TEST_REALIZED.java
The method setNullTestLastTZ() is undefined for the type TILDA__TEST_REALIZED TILDA__TEST_REALIZED.java
The method setTestFirstTZ(String) is undefined for the type TILDA__TEST_REALIZED TILDA__TEST_REALIZED.java
The method setTestLastTZ(String) is undefined for the type TILDA__TEST_REALIZED TILDA__TEST_REALIZED.java
ldhasson added a commit that referenced this issue Oct 30, 2019
@ldhasson
#429 check compatibility of aggregate for Datetime column
af261f2
ldhasson added a commit that referenced this issue Oct 30, 2019
@ldhasson
#429 improper aggregate checks during validation
0f9e5e5 
We have to check generically for all types, not just Datetime.
ldhasson added a commit that referenced this issue Oct 30, 2019
@ldhasson
Merge pull request #459 from CapsicoHealth/#429_improper_aggregate_ch…
e5244c9 
…eck_for_datetime_columns

#429 improper aggregate check for datetime columns
@ldhasson ldhasson reopened this Nov 7, 2019
ldhasson added a commit that referenced this issue Nov 7, 2019
@ldhasson
#429 code cleanup and fix for missing break for min/max datetime
8b6e1e8
ldhasson added a commit that referenced this issue Nov 10, 2019
@ldhasson
#429 gotta handle aggregates for TZ cols as well
affd992
ldhasson added a commit that referenced this issue Nov 10, 2019
@ldhasson
#429 updated manifest template
c58ca33
ldhasson added a commit that referenced this issue Nov 11, 2019
@ldhasson
Merge pull request #469 from CapsicoHealth/#429_improper_aggregate_ch…
1e3d5de 
…eck_for_datetime_columns

#429 improper aggregate check for datetime columns
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug S1
Projects
Tilda - Main
Archived in project
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ldhasson