Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: error getting partitionQuery from table.partition #14369

Merged
merged 1 commit into from
Apr 29, 2021

Conversation

eriendeng
Copy link
Contributor

@eriendeng eriendeng commented Apr 27, 2021

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

  • Has associated issue: closes bug: cannot get partition query #14368
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

Copy link
Member

@villebro villebro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@codecov
Copy link

codecov bot commented Apr 29, 2021

Codecov Report

Merging #14369 (7e9d078) into master (9156f08) will decrease coverage by 0.70%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #14369      +/-   ##
==========================================
- Coverage   77.83%   77.13%   -0.71%     
==========================================
  Files         934     1004      +70     
  Lines       47320    52301    +4981     
  Branches     5913     7551    +1638     
==========================================
+ Hits        36831    40340    +3509     
- Misses      10346    11817    +1471     
- Partials      143      144       +1     
Flag Coverage Δ
javascript 72.03% <ø> (+7.40%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...et-frontend/src/SqlLab/components/TableElement.jsx 55.39% <0.00%> (-32.98%) ⬇️
...frontend/src/components/WarningIconWithTooltip.tsx 0.00% <0.00%> (-88.89%) ⬇️
...Control/AdhocFilterEditPopoverSimpleTabContent.jsx 3.33% <0.00%> (-71.51%) ⬇️
superset-frontend/src/common/components/Fade.tsx 33.33% <0.00%> (-66.67%) ⬇️
...plore/components/controls/FilterBoxItemControl.jsx 13.20% <0.00%> (-60.38%) ⬇️
...frontend/src/SqlLab/components/QueryStateLabel.jsx 40.00% <0.00%> (-60.00%) ⬇️
...terControl/AdhocFilterEditPopoverSqlTabContent.jsx 6.66% <0.00%> (-56.67%) ⬇️
...-frontend/src/visualizations/presets/MainPreset.js 50.00% <0.00%> (-50.00%) ⬇️
...trols/DndColumnSelectControl/components/Option.tsx 0.00% <0.00%> (-50.00%) ⬇️
.../controls/FilterControl/AdhocFilterEditPopover.jsx 10.00% <0.00%> (-48.00%) ⬇️
... and 279 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9156f08...7e9d078. Read the comment docs.

@villebro villebro merged commit 2dc3fa3 into apache:master Apr 29, 2021
@villebro
Copy link
Member

This regression was introduced in the refactor done in #10867 - I did a quick review of it and didn't find any other similar regressions.

@villebro villebro added the v1.2 label Apr 29, 2021
amitmiran137 pushed a commit that referenced this pull request Apr 30, 2021
Co-authored-by: eriendeng <eriendeng@tencent.com>
(cherry picked from commit 2dc3fa3)
amitmiran137 pushed a commit to nielsen-oss/superset that referenced this pull request May 3, 2021
* master: (38 commits)
  refactor(native-filters): allow cascading only for filter_select (apache#14441)
  test(maximize-chart): Add tests to maximize chart action (apache#14371)
  fix: fixing mysql error message (apache#14416)
  feat: Logic added to limiting factor column in Query model (apache#13521)
  change relationship (apache#14435)
  fix: bootstrap data permissions (apache#14348)
  fix: parse simple string error message values (apache#14360)
  chore: add stack trace to all calls of logger.error (apache#14382)
  update README with new docs and recordings (apache#14432)
  Renamed impyla from implya in impala.mdx and Renamed PIP package impyla from impala in index.mdx (apache#14425)
  fix(native-filters): fix filter scope error (apache#14426)
  feat: Adding limiting_factor column to Query model (apache#14234)
  feat: Add etag caching to dashboard APIs (apache#14357)
  chore: Moves Card to the components folder (apache#14139)
  feat: Dynamic imports for the Icons component (apache#14318)
  feat: Support env vars configuration for WebSocket server (apache#14398)
  fix: SQLLab role permissions (apache#14372)
  fix(native-filters): always show filters without dataset (apache#14409)
  fix error getting partitionQuery from table.partition (apache#14369)
  refactor: Boostrap to AntD - Tabs (apache#14048)
  ...
QAlexBall pushed a commit to QAlexBall/superset that referenced this pull request Dec 29, 2021
Co-authored-by: eriendeng <eriendeng@tencent.com>
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.2.0 labels Mar 12, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/XS v1.2 🚢 1.2.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: cannot get partition query
3 participants