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.

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

Unexpected exception for json_extract: None const type of json path argument of function json_extract, e.what() = DB::Exception, #8510

Closed
SeaRise opened this issue Dec 13, 2023 · 3 comments · Fixed by #8555
Labels
component/compute severity/minor type/bug The issue is confirmed as a bug.

Comments

@SeaRise
Copy link
Contributor

SeaRise commented Dec 13, 2023

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

drop table if exists test.t_json;
create table test.t_json(col_json json);
alter table test.t_json set tiflash replica 1;
insert into test.t_json values(null), ('[]'), ('{}'), ('{\"a\":{\"c\":{}, \"d\":{}}, \"b\":{\"e\":{}, \"f\":{}}}'); #NO_UNESCAPE

drop table if exists test.t_path;
create table test.t_path(col_path varchar(100));
alter table test.t_path set tiflash replica 1;
insert into test.t_path values(null), ('$'), ('\$[0]'), ('\$[1]'), ('\$.a'), ('\$.b'); #NO_UNESCAPE

set tidb_allow_mpp=1;
set tidb_enforce_mpp=1; 
set tidb_isolation_read_engines='tiflash'; 
select col_json, col_path, json_extract(col_json, col_path) as res from (select * from test.t_json join test.t_path) t order by col_json, col_path;

2. What did you expect to see? (Required)

query success

3. What did you see instead (Required)

ERROR 1105 (HY000): other error for mpp stream: Code: 43, e.displayText() = DB::Exception: None const type of json path argument of function json_extract, e.what() = DB::Exception,

4. What is your TiFlash version? (Required)

8.0.11-TiDB-v7.6.0-alpha

@SeaRise SeaRise added the type/bug The issue is confirmed as a bug. label Dec 13, 2023
@SeaRise
Copy link
Contributor Author

SeaRise commented Dec 13, 2023

/cc @yibin87

@yibin87
Copy link
Contributor

yibin87 commented Dec 13, 2023

Yeah, it is expected since it's very corner case and doesn't make much sense to use column as path arguments.

@SeaRise
Copy link
Contributor Author

SeaRise commented Dec 13, 2023

Okay, so the severity is set to minor level.

@ti-chi-bot ti-chi-bot bot closed this as completed in #8555 Jan 9, 2024
@ti-chi-bot ti-chi-bot bot closed this as completed in e317f2a Jan 9, 2024
wuhuizuo pushed a commit to PingCAP-QE/artifacts that referenced this issue May 20, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [pingcap/tiflash](https://github.com/pingcap/tiflash) | major |
`v7.6.0` -> `v8.0.0` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>pingcap/tiflash (pingcap/tiflash)</summary>

###
[`v8.0.0`](https://github.com/pingcap/tiflash/releases/tag/v8.0.0):
TiFlash v8.0.0

[Compare
Source](https://github.com/pingcap/tiflash/compare/v7.6.0...v8.0.0)

For new features, improvements, and bug fixes released in 8.0.0 for
tiflash, see [TiDB 8.0.0 release
notes](https://docs.pingcap.com/tidb/v8.0/release-8.0.0/).
See the difference from the issue perspective: <details>

    - pingcap/tiflash#8510

-   [#&#8203;8382](https://github.com/pingcap/tiflash/issues/8382)
-   [#&#8203;8697](https://github.com/pingcap/tiflash/issues/8697)
-   [#&#8203;8685](https://github.com/pingcap/tiflash/issues/8685)
-   [#&#8203;8703](https://github.com/pingcap/tiflash/issues/8703)
-   [#&#8203;8712](https://github.com/pingcap/tiflash/issues/8712)
-   [#&#8203;8715](https://github.com/pingcap/tiflash/issues/8715)
-   [#&#8203;8710](https://github.com/pingcap/tiflash/issues/8710)
-   [#&#8203;8419](https://github.com/pingcap/tiflash/issues/8419)
-   [#&#8203;8695](https://github.com/pingcap/tiflash/issues/8695)
-   [#&#8203;8688](https://github.com/pingcap/tiflash/issues/8688)
-   [#&#8203;8351](https://github.com/pingcap/tiflash/issues/8351)
- [#&#8203;8323](https://github.com/pingcap/tiflash/issues/8323),
close [#&#8203;8504](https://github.com/pingcap/tiflash/issues/8504)
-   [#&#8203;8351](https://github.com/pingcap/tiflash/issues/8351)
-   [#&#8203;8711](https://github.com/pingcap/tiflash/issues/8711)
-   [#&#8203;8351](https://github.com/pingcap/tiflash/issues/8351)
-   [#&#8203;8751](https://github.com/pingcap/tiflash/issues/8751)
-   [#&#8203;8674](https://github.com/pingcap/tiflash/issues/8674)
-   [#&#8203;8765](https://github.com/pingcap/tiflash/issues/8765)
-   [#&#8203;8768](https://github.com/pingcap/tiflash/issues/8768)
-   [pingcap/tidb#51204](https://github.com/pingcap/tidb/issues/51204)
-   [#&#8203;8777](https://github.com/pingcap/tiflash/issues/8777)
-   [#&#8203;8691](https://github.com/pingcap/tiflash/issues/8691)
-   [#&#8203;8796](https://github.com/pingcap/tiflash/issues/8796)
-   [#&#8203;8799](https://github.com/pingcap/tiflash/issues/8799)
-   [#&#8203;8806](https://github.com/pingcap/tiflash/issues/8806)
-   [#&#8203;8824](https://github.com/pingcap/tiflash/issues/8824)
-   [#&#8203;8826](https://github.com/pingcap/tiflash/issues/8826)
-   [#&#8203;8843](https://github.com/pingcap/tiflash/issues/8843)
-   [#&#8203;8837](https://github.com/pingcap/tiflash/issues/8837)
-   [#&#8203;8589](https://github.com/pingcap/tiflash/issues/8589)

      </details>

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/PingCAP-QE/artifacts).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yNjkuMiIsInVwZGF0ZWRJblZlciI6IjM3LjI2OS4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/compute severity/minor type/bug The issue is confirmed as a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants