You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "Edward-Gavin (via GitHub)" <gi...@apache.org> on 2024/11/05 03:56:58 UTC

[PR] [doc] Update expire-partition.md & procedures.md [paimon]

Edward-Gavin opened a new pull request, #4452:
URL: https://github.com/apache/paimon/pull/4452

   <!-- Please specify the module before the PR name: [core] ... or [flink] ... -->
   
   ### Purpose
   
   <!-- Linking this pull request to the issue -->
   Linked issue: **no**
   
   <!-- What is the purpose of the change -->
   
   ### Tests
   
   <!-- List UT and IT cases to verify this change -->
   **no**
   
   ### API and Format
   
   <!-- Does this change affect API or storage format -->
   **no**
   ### Documentation
   Update `docs/content/flink/procedures.md` and `docs/content/flink/expire-partition.md`.
   
   <!-- Does this change introduce a new feature -->
   


-- 
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-unsubscribe@paimon.apache.org

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

Re: [PR] [doc] Update expire-partition.md & procedures.md [paimon]

Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on code in PR #4452:
URL: https://github.com/apache/paimon/pull/4452#discussion_r1828743571


##########
docs/content/flink/procedures.md:
##########
@@ -39,8 +39,8 @@ argument can be omitted. For the above example, the call statement is \
 ``CALL sys.compact(`table` => 'default.t', options => 'sink.parallelism=4')``.
 
 Specify partitions: we use string to represent partition filter. "," means "AND" and ";" means "OR". For example, if you want
-to specify two partitions date=01 and date=02, you need to write 'date=01;date=02'; If you want to specify one partition
-with date=01 and day=01, you need to write 'date=01,day=01'.
+to specify two partitions date=01 and date=02, you need to write 'date=01,date=02'; If you want to specify one partition
+with date=01 and day=01, you need to write 'date=01;day=01'.

Review Comment:
   I think you are wrong..



-- 
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-unsubscribe@paimon.apache.org

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

Re: [PR] [doc] Update expire-partition.md & procedures.md [paimon]

Posted by "Edward-Gavin (via GitHub)" <gi...@apache.org>.
Edward-Gavin closed pull request #4452: [doc] Update expire-partition.md & procedures.md
URL: https://github.com/apache/paimon/pull/4452


-- 
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-unsubscribe@paimon.apache.org

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

Re: [PR] [doc] Update expire-partition.md & procedures.md [paimon]

Posted by "Edward-Gavin (via GitHub)" <gi...@apache.org>.
Edward-Gavin commented on code in PR #4452:
URL: https://github.com/apache/paimon/pull/4452#discussion_r1828747713


##########
docs/content/flink/procedures.md:
##########
@@ -39,8 +39,8 @@ argument can be omitted. For the above example, the call statement is \
 ``CALL sys.compact(`table` => 'default.t', options => 'sink.parallelism=4')``.
 
 Specify partitions: we use string to represent partition filter. "," means "AND" and ";" means "OR". For example, if you want
-to specify two partitions date=01 and date=02, you need to write 'date=01;date=02'; If you want to specify one partition
-with date=01 and day=01, you need to write 'date=01,day=01'.
+to specify two partitions date=01 and date=02, you need to write 'date=01,date=02'; If you want to specify one partition
+with date=01 and day=01, you need to write 'date=01;day=01'.

Review Comment:
   Sorry, it's my fault.



-- 
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-unsubscribe@paimon.apache.org

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