You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/09/27 20:30:59 UTC

[GitHub] [accumulo] milleruntime opened a new issue, #2974: Fate Admin print doesn't filter on status

milleruntime opened a new issue, #2974:
URL: https://github.com/apache/accumulo/issues/2974

   The status filter on the new Admin fate command does not change the results:
   <pre>
   04:26:21 {main} ~/workspace/uno/install/logs/accumulo$ accumulo admin fate -p -s IN_PROGRESS
   2022-09-27T16:26:29,795 [conf.SiteConfiguration] INFO : Found Accumulo configuration on classpath at /home/mpmill4/workspace/uno/install/accumulo-2.1.0-SNAPSHOT/conf/accumulo.properties
   2022-09-27T16:26:31,085 [metrics.MetricsUtil] INFO : Metric producer PropStoreMetrics initialize
   TABLE_BULK_IMPORT2 txid: 6324179035588914  status: SUBMITTED          locked: []              locking: [R:3]           op: PrepBulkImport  created: 2022-09-27T20:25:40.716Z
   TABLE_COMPACT   txid: 4d7bf30c0676eb5d  status: SUBMITTED          locked: [R:+default]    locking: [R:3]           op: CompactRange    created: 2022-09-27T20:26:09.732Z
   TABLE_BULK_IMPORT2 txid: 0176dfaebcec9591  status: SUBMITTED          locked: []              locking: [R:3]           op: PrepBulkImport  created: 2022-09-27T20:26:10.648Z
   TABLE_MERGE     txid: 1eeba54e6e769e7c  status: IN_PROGRESS        locked: [R:+default, W:3] locking: []              op: TableRangeOpWait created: 2022-09-27T20:25:39.58Z
   TABLE_COMPACT   txid: 09760433eb43a5ce  status: IN_PROGRESS        locked: [R:+default, R:1] locking: []              op: CompactionDriver created: 2022-09-27T20:26:00.977Z
   TABLE_MERGE     txid: 37d000bf5948ab30  status: SUBMITTED          locked: [R:+default]    locking: [W:1]           op: TableRangeOp    created: 2022-09-27T20:26:25.915Z
   </pre>


-- 
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: notifications-unsubscribe@accumulo.apache.org.apache.org

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


[GitHub] [accumulo] ctubbsii commented on issue #2974: Fate Admin print doesn't filter on status

Posted by GitBox <gi...@apache.org>.
ctubbsii commented on issue #2974:
URL: https://github.com/apache/accumulo/issues/2974#issuecomment-1263652635

   Do we really need this? I feel like a user can just use `grep` themselves, if each item is already printed on separate lines.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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


[GitHub] [accumulo] DomGarguilo commented on issue #2974: Fate Admin print doesn't filter on status

Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on issue #2974:
URL: https://github.com/apache/accumulo/issues/2974#issuecomment-1262757743

   It looks like the filter is not getting picked up properly and for some reason the value is null by the time the filtering actually occurs.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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


[GitHub] [accumulo] milleruntime closed issue #2974: Fate Admin print doesn't filter on status

Posted by GitBox <gi...@apache.org>.
milleruntime closed issue #2974: Fate Admin print doesn't filter on status
URL: https://github.com/apache/accumulo/issues/2974


-- 
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: notifications-unsubscribe@accumulo.apache.org

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


[GitHub] [accumulo] EdColeman commented on issue #2974: Fate Admin print doesn't filter on status

Posted by GitBox <gi...@apache.org>.
EdColeman commented on issue #2974:
URL: https://github.com/apache/accumulo/issues/2974#issuecomment-1262311511

   The `-t` option is used for specifying table in a number of commands - would it be less confusing is that was sort of reserved for that?  The fate command does not take a table so it is unambiguous here, so the primary consideration would be clarity. `-s` for status seems a little more natural the `-t` for status type, but either way seems okay.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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


[GitHub] [accumulo] milleruntime commented on issue #2974: Fate Admin print doesn't filter on status

Posted by GitBox <gi...@apache.org>.
milleruntime commented on issue #2974:
URL: https://github.com/apache/accumulo/issues/2974#issuecomment-1262339810

   Yeah I like `-s` better, like Ed said. So we can just change that one. But this ticket was more for making sure the filter actually works.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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


[GitHub] [accumulo] milleruntime commented on issue #2974: Fate Admin print doesn't filter on status

Posted by GitBox <gi...@apache.org>.
milleruntime commented on issue #2974:
URL: https://github.com/apache/accumulo/issues/2974#issuecomment-1260736769

   I think we should also make the options the same. For example, Admin uses `fate -p -s IN_PROGRESS` while the Shell command uses `-t` for filtering on status Type.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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


[GitHub] [accumulo] milleruntime commented on issue #2974: Fate Admin print doesn't filter on status

Posted by GitBox <gi...@apache.org>.
milleruntime commented on issue #2974:
URL: https://github.com/apache/accumulo/issues/2974#issuecomment-1263501709

   > It looks like the filter is not getting picked up properly and for some reason the value is null by the time the filtering actually occurs.
   
   Right. I have a simple fix ready to go but was waiting on #2976 since that is also breaking the command.


-- 
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: notifications-unsubscribe@accumulo.apache.org

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


[GitHub] [accumulo] milleruntime commented on issue #2974: Fate Admin print doesn't filter on status

Posted by GitBox <gi...@apache.org>.
milleruntime commented on issue #2974:
URL: https://github.com/apache/accumulo/issues/2974#issuecomment-1263670236

   > Do we really need this? I feel like a user can just use `grep` themselves, if each item is already printed on separate lines.
   
   Not really. Now that its in Admin you can easily pipe it to grep. The only thing that is kinda nice is that the description also prints all the possible types:
   <pre>
             -s, --state
               <state>[ -s <state>...] Print transactions in the state(s) {NEW, 
               IN_PROGRESS, FAILED_IN_PROGRESS, FAILED, SUCCESSFUL}
               Default: []
   </pre>


-- 
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: notifications-unsubscribe@accumulo.apache.org

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


[GitHub] [accumulo] DomGarguilo commented on issue #2974: Fate Admin print doesn't filter on status

Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on issue #2974:
URL: https://github.com/apache/accumulo/issues/2974#issuecomment-1261376900

   > make the options the same. For example, Admin uses `fate -p -s IN_PROGRESS` while the Shell command uses `-t` for filtering on status Type.
   
   which option do you think is better?


-- 
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: notifications-unsubscribe@accumulo.apache.org

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