Re: [PR] PHOENIX-7313 All cell versions should not be retained during flushes … [phoenix]

2024-05-11 Thread via GitHub


virajjasani commented on PR #1888:
URL: https://github.com/apache/phoenix/pull/1888#issuecomment-2106005182

   First build timed out after ~9 hr, and second one ran for only ~16 min.
   Triggered new one: 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1888/3/


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@phoenix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] PHOENIX-628 Support native JSON data type [phoenix]

2024-05-11 Thread via GitHub


virajjasani merged PR #1780:
URL: https://github.com/apache/phoenix/pull/1780


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@phoenix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[PR] Bump com.jayway.jsonpath:json-path from 2.6.0 to 2.9.0 in /phoenix-core-client [phoenix]

2024-05-11 Thread via GitHub


dependabot[bot] opened a new pull request, #1890:
URL: https://github.com/apache/phoenix/pull/1890

   Bumps [com.jayway.jsonpath:json-path](https://github.com/jayway/JsonPath) 
from 2.6.0 to 2.9.0.
   
   Release notes
   Sourced from https://github.com/jayway/JsonPath/releases";>com.jayway.jsonpath:json-path's
 releases.
   
   json-path-2.9.0
   What's Changed
   
   Fix for https://github.com/advisories/GHSA-pfh2-hfmq-phg5";>CVE-2023-51074.
   update dependencies by https://github.com/SingingBush";>@​SingingBush in https://redirect.github.com/json-path/JsonPath/pull/965";>json-path/JsonPath#965
   JPMS: define Automatic-Module-Name as json.path by https://github.com/SingingBush";>@​SingingBush in https://redirect.github.com/json-path/JsonPath/pull/966";>json-path/JsonPath#966
   Bump json-smart version from 2.4.10 to 2.5.0 by https://github.com/shoothzj";>@​shoothzj in https://redirect.github.com/json-path/JsonPath/pull/945";>json-path/JsonPath#945
   Fixed rendering error on $..book[?(@.price <= 
$['expensive'])] in README.md by https://github.com/lulunac27a";>@​lulunac27a in https://redirect.github.com/json-path/JsonPath/pull/967";>json-path/JsonPath#967
   [build] Remove deprecated gradle convetion usage by https://github.com/shoothzj";>@​shoothzj in https://redirect.github.com/json-path/JsonPath/pull/946";>json-path/JsonPath#946
   Check for the existence of the next significant bracket by https://github.com/twobiers";>@​twobiers in https://redirect.github.com/json-path/JsonPath/pull/985";>json-path/JsonPath#985
   Update versions by https://github.com/kallestenflo";>@​kallestenflo in https://redirect.github.com/json-path/JsonPath/pull/987";>json-path/JsonPath#987
   
   New Contributors
   
   https://github.com/lulunac27a";>@​lulunac27a 
made their first contribution in https://redirect.github.com/json-path/JsonPath/pull/967";>json-path/JsonPath#967
   https://github.com/twobiers";>@​twobiers made 
their first contribution in https://redirect.github.com/json-path/JsonPath/pull/985";>json-path/JsonPath#985
   
   Full Changelog: https://github.com/json-path/JsonPath/compare/json-path-2.8.0...json-path-2.9.0";>https://github.com/json-path/JsonPath/compare/json-path-2.8.0...json-path-2.9.0
   json-path-2.8.0
   Upgrade json-smart to fix https://www.cve.org/CVERecord?id=CVE-2023-1370";>https://www.cve.org/CVERecord?id=CVE-2023-1370
   json-path-2.7.0
   No release notes provided.
   
   
   
   Commits
   
   https://github.com/json-path/JsonPath/commit/af7e516c69df680a6584fca7180ef082eb67c96c";>af7e516
 Release 2.9.0
   https://github.com/json-path/JsonPath/commit/af4dfcc17a9cb69002a5abc6e88a7745aef4c43d";>af4dfcc
 Make PropertyPathToken public, closes https://redirect.github.com/jayway/JsonPath/issues/955";>#955
   https://github.com/json-path/JsonPath/commit/49b1151b11fbb147382c9c0f1c898b1ca0d2d3ec";>49b1151
 Update versions (https://redirect.github.com/jayway/JsonPath/issues/987";>#987)
   https://github.com/json-path/JsonPath/commit/71a09c1193726c010917f1157ecbb069ad6c3e3b";>71a09c1
 Check for the existence of the next significant bracket (https://redirect.github.com/jayway/JsonPath/issues/985";>#985)
   https://github.com/json-path/JsonPath/commit/900ebfe58f4c9e78b209401c642320ef332ba618";>900ebfe
 Remove deprecated gradle usage (https://redirect.github.com/jayway/JsonPath/issues/946";>#946)
   https://github.com/json-path/JsonPath/commit/946274d3cb2ee51098bc0d8ba767a01da525218f";>946274d
 Fixed rendering error on $..book[?(@.price <= $['expensive'])] 
in README.md...
   https://github.com/json-path/JsonPath/commit/425bcb18a5ffdfafcc60190c892f941ea4117fb7";>425bcb1
 Bump json-smart version from 2.4.10 to 2.5.0 (https://redirect.github.com/jayway/JsonPath/issues/945";>#945)
   https://github.com/json-path/JsonPath/commit/2d57ab361512451fc3352816648357ffbd5c2a13";>2d57ab3
 JPMS: define Automatic-Module-Name as json.path (https://redirect.github.com/jayway/JsonPath/issues/966";>#966)
   https://github.com/json-path/JsonPath/commit/1a57f7800b2c101b6fca218c3f04a8b6adf04122";>1a57f78
 update dependencies (https://redirect.github.com/jayway/JsonPath/issues/965";>#965)
   https://github.com/json-path/JsonPath/commit/21de6207945dfca6b49809fa9dd08c3781b7ab9d";>21de620
 Prepare next version
   Additional commits viewable in https://github.com/jayway/JsonPath/compare/json-path-2.6.0...json-path-2.9.0";>compare
 view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=com.jayway.jsonpath:json-path&package-manager=maven&previous-version=2.6.0&new-version=2.9.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (d

[PR] Bump com.jayway.jsonpath:json-path from 2.6.0 to 2.9.0 in /phoenix-core [phoenix]

2024-05-11 Thread via GitHub


dependabot[bot] opened a new pull request, #1889:
URL: https://github.com/apache/phoenix/pull/1889

   Bumps [com.jayway.jsonpath:json-path](https://github.com/jayway/JsonPath) 
from 2.6.0 to 2.9.0.
   
   Release notes
   Sourced from https://github.com/jayway/JsonPath/releases";>com.jayway.jsonpath:json-path's
 releases.
   
   json-path-2.9.0
   What's Changed
   
   Fix for https://github.com/advisories/GHSA-pfh2-hfmq-phg5";>CVE-2023-51074.
   update dependencies by https://github.com/SingingBush";>@​SingingBush in https://redirect.github.com/json-path/JsonPath/pull/965";>json-path/JsonPath#965
   JPMS: define Automatic-Module-Name as json.path by https://github.com/SingingBush";>@​SingingBush in https://redirect.github.com/json-path/JsonPath/pull/966";>json-path/JsonPath#966
   Bump json-smart version from 2.4.10 to 2.5.0 by https://github.com/shoothzj";>@​shoothzj in https://redirect.github.com/json-path/JsonPath/pull/945";>json-path/JsonPath#945
   Fixed rendering error on $..book[?(@.price <= 
$['expensive'])] in README.md by https://github.com/lulunac27a";>@​lulunac27a in https://redirect.github.com/json-path/JsonPath/pull/967";>json-path/JsonPath#967
   [build] Remove deprecated gradle convetion usage by https://github.com/shoothzj";>@​shoothzj in https://redirect.github.com/json-path/JsonPath/pull/946";>json-path/JsonPath#946
   Check for the existence of the next significant bracket by https://github.com/twobiers";>@​twobiers in https://redirect.github.com/json-path/JsonPath/pull/985";>json-path/JsonPath#985
   Update versions by https://github.com/kallestenflo";>@​kallestenflo in https://redirect.github.com/json-path/JsonPath/pull/987";>json-path/JsonPath#987
   
   New Contributors
   
   https://github.com/lulunac27a";>@​lulunac27a 
made their first contribution in https://redirect.github.com/json-path/JsonPath/pull/967";>json-path/JsonPath#967
   https://github.com/twobiers";>@​twobiers made 
their first contribution in https://redirect.github.com/json-path/JsonPath/pull/985";>json-path/JsonPath#985
   
   Full Changelog: https://github.com/json-path/JsonPath/compare/json-path-2.8.0...json-path-2.9.0";>https://github.com/json-path/JsonPath/compare/json-path-2.8.0...json-path-2.9.0
   json-path-2.8.0
   Upgrade json-smart to fix https://www.cve.org/CVERecord?id=CVE-2023-1370";>https://www.cve.org/CVERecord?id=CVE-2023-1370
   json-path-2.7.0
   No release notes provided.
   
   
   
   Commits
   
   https://github.com/json-path/JsonPath/commit/af7e516c69df680a6584fca7180ef082eb67c96c";>af7e516
 Release 2.9.0
   https://github.com/json-path/JsonPath/commit/af4dfcc17a9cb69002a5abc6e88a7745aef4c43d";>af4dfcc
 Make PropertyPathToken public, closes https://redirect.github.com/jayway/JsonPath/issues/955";>#955
   https://github.com/json-path/JsonPath/commit/49b1151b11fbb147382c9c0f1c898b1ca0d2d3ec";>49b1151
 Update versions (https://redirect.github.com/jayway/JsonPath/issues/987";>#987)
   https://github.com/json-path/JsonPath/commit/71a09c1193726c010917f1157ecbb069ad6c3e3b";>71a09c1
 Check for the existence of the next significant bracket (https://redirect.github.com/jayway/JsonPath/issues/985";>#985)
   https://github.com/json-path/JsonPath/commit/900ebfe58f4c9e78b209401c642320ef332ba618";>900ebfe
 Remove deprecated gradle usage (https://redirect.github.com/jayway/JsonPath/issues/946";>#946)
   https://github.com/json-path/JsonPath/commit/946274d3cb2ee51098bc0d8ba767a01da525218f";>946274d
 Fixed rendering error on $..book[?(@.price <= $['expensive'])] 
in README.md...
   https://github.com/json-path/JsonPath/commit/425bcb18a5ffdfafcc60190c892f941ea4117fb7";>425bcb1
 Bump json-smart version from 2.4.10 to 2.5.0 (https://redirect.github.com/jayway/JsonPath/issues/945";>#945)
   https://github.com/json-path/JsonPath/commit/2d57ab361512451fc3352816648357ffbd5c2a13";>2d57ab3
 JPMS: define Automatic-Module-Name as json.path (https://redirect.github.com/jayway/JsonPath/issues/966";>#966)
   https://github.com/json-path/JsonPath/commit/1a57f7800b2c101b6fca218c3f04a8b6adf04122";>1a57f78
 update dependencies (https://redirect.github.com/jayway/JsonPath/issues/965";>#965)
   https://github.com/json-path/JsonPath/commit/21de6207945dfca6b49809fa9dd08c3781b7ab9d";>21de620
 Prepare next version
   Additional commits viewable in https://github.com/jayway/JsonPath/compare/json-path-2.6.0...json-path-2.9.0";>compare
 view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=com.jayway.jsonpath:json-path&package-manager=maven&previous-version=2.6.0&new-version=2.9.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (d

[jira] [Commented] (PHOENIX-628) Support native JSON data type

2024-05-11 Thread Viraj Jasani (Jira)


[ 
https://issues.apache.org/jira/browse/PHOENIX-628?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17845643#comment-17845643
 ] 

Viraj Jasani commented on PHOENIX-628:
--

As we don't have any System table schema changes, we can target backport to 5.2 
branch as well. 5.2.1 release can also enjoy this feature.

> Support native JSON data type
> -
>
> Key: PHOENIX-628
> URL: https://issues.apache.org/jira/browse/PHOENIX-628
> Project: Phoenix
>  Issue Type: New Feature
>Affects Versions: 5.1.4
>Reporter: James R. Taylor
>Assignee: Ranganath Govardhanagiri
>  Labels: JSON, Java, SQL
> Fix For: 4.4.1, 5.3.0
>
> Attachments: JSON Support for Phoenix.docx, Screen Shot 2022-02-02 at 
> 12.23.24 PM.png, image-2023-12-07-11-26-56-198.png
>
>
> MongoDB and Postgres do some interesting things with JSON. We should look at 
> adding similar support. For a detailed description, see JSONB support in 
> Postgres: 
> [http://www.craigkerstiens.com/2014/03/24/Postgres-9.4-Looking-up]
> [http://www.depesz.com/2014/03/25/waiting-for-9-4-introduce-jsonb-a-structured-format-for-storing-json/]
> [http://michael.otacoo.com/postgresql-2/manipulating-jsonb-data-with-key-unique/]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Commented] (PHOENIX-628) Support native JSON data type

2024-05-11 Thread Viraj Jasani (Jira)


[ 
https://issues.apache.org/jira/browse/PHOENIX-628?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17845645#comment-17845645
 ] 

Viraj Jasani commented on PHOENIX-628:
--

Once 5.2 backport PR is ready, I will reopen this Jira to update the fix 
version. We can move additional follow-up support (e.g. JSON_MODIFY as Atomic 
Upsert) as separate Jira tasks and link to this Jira.

> Support native JSON data type
> -
>
> Key: PHOENIX-628
> URL: https://issues.apache.org/jira/browse/PHOENIX-628
> Project: Phoenix
>  Issue Type: New Feature
>Affects Versions: 5.1.4
>Reporter: James R. Taylor
>Assignee: Ranganath Govardhanagiri
>Priority: Blocker
>  Labels: JSON, Java, SQL
> Fix For: 4.4.1, 5.3.0
>
> Attachments: JSON Support for Phoenix.docx, Screen Shot 2022-02-02 at 
> 12.23.24 PM.png, image-2023-12-07-11-26-56-198.png
>
>
> MongoDB and Postgres do some interesting things with JSON. We should look at 
> adding similar support. For a detailed description, see JSONB support in 
> Postgres: 
> [http://www.craigkerstiens.com/2014/03/24/Postgres-9.4-Looking-up]
> [http://www.depesz.com/2014/03/25/waiting-for-9-4-introduce-jsonb-a-structured-format-for-storing-json/]
> [http://michael.otacoo.com/postgresql-2/manipulating-jsonb-data-with-key-unique/]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)