This repository has been archived by the owner on Aug 2, 2022. It is now read-only.
Merge develop to master for ODFE 1.10.1.0 release #733
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…on (#687) * add interval type * add interval type * added docs * update * update * corrected typo * update * added interval in sql parser * update
* Change grammar and ast builder * Define index symbol and resolve it when visiting qualified name * Raise error if qualifier is not index name or alias * Fix index not found exception by adding another get name or alias method * Remove qualifier if select item is a qualified field * Prepare PR * Prepare PR * Add doctest * Add bug fix test * Add more UT for where after where PR merged
* Add PPL stats endpoint * update
…string type to date and time types (#694) * added datetime type, modified date, time, timestamp types, added tests * added test * update * update * Added java doc * update * update * update * update * update * update * update * update
* add serialization support for expression * update: * add branch coverage for core engine * address comments
…ws (#722) * update user docs * fix typo * fix link * Update sql-odbc/docs/user/windows_configure_dsn.md Co-authored-by: Jordan Wilson <37088125+jordanw-bq@users.noreply.github.com> * Update sql-odbc/docs/user/windows_configure_dsn.md Co-authored-by: Jordan Wilson <37088125+jordanw-bq@users.noreply.github.com> * Update sql-odbc/docs/user/windows_configure_dsn.md Co-authored-by: Jordan Wilson <37088125+jordanw-bq@users.noreply.github.com> * add fetch size to configuration options * add a note that host should be prefaced with 'https://` for encrypted connections Co-authored-by: Jordan Wilson <37088125+jordanw-bq@users.noreply.github.com> Co-authored-by: Rupal Mahajan <>
* ast - rare and top command * fix build failure * fix build failure - test coverage ratio for ppl * add logical, physical plan for rare & top * merge rare & top operator since both commands operate identical to each other * add tests * update docs * add comments * addressing PR comments * fix build error * address PR comments * address PR comments: - remove list - use generic getGroupByList function for stats - use generic getFieldList function for dedup * use interface instead of specific type for input of find() * update doc * replace rareTopFlag with enum Co-authored-by: Rupal Mahajan <>
* Bump to odfe 1.10.1, update release notes * Fix version number and release notes
chloe-zh
approved these changes
Sep 8, 2020
anirudha
approved these changes
Sep 8, 2020
* Change grammar and AST builder * Change analyzer * Expression name issue * Add context in sql ast builder * Add javadoc, deduplicate and dedicate aggregation builder * Add UT for aggregation builder * Add validation in aggregation builder * Fix function arguments are not on child list issue * Add validation for non aggregated column if no group by * Fix jacoco * Refactor and add UT * Add comparison test * Fix broken UT * Fix UT * Add IT for validation * Fix broken IT * Add more comparison tests * Add more comparison tests * Add comparison test for bug fix 280 * Remove COUNT and change grammar for SUM/AVG for now * Add comparison test for bug fix 288 * Support group by alias * Support group by ordinal and add comparison test * Add UT for edge case of ordinal * Pass jacoco * Skip problematic IT for now * Add more comparison test for index alias * Prepare PR * Add and reformat javadoc * Refactor query spec by separating visitor * Prepare PR * Bug fix 373 * Bug fix 717
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.