[GitHub] [arrow] houqp commented on a change in pull request #7306: ARROW-3688: [Rust] Add append_values for primitive builders

2020-05-30 Thread GitBox


houqp commented on a change in pull request #7306:
URL: https://github.com/apache/arrow/pull/7306#discussion_r432907904



##
File path: rust/arrow/src/array/builder.rs
##
@@ -500,6 +500,49 @@ impl PrimitiveBuilder {
 Ok(())
 }
 
+/// Appends values from a slice of type `T` and a validity byte slice
+pub fn append_values(
+ self,
+values: &[T::Native],
+is_valid: &[u8],
+offset: usize,
+) -> Result<()> {
+let ceil = bit_util::round_upto_power_of_2(values.len(), 8);
+if offset == 0 {
+// if there is no offset, the value slice should align with 
validity
+assert_eq!(
+ceil / 8,
+is_valid.len(),
+"value slice not aligned with validity slice"
+);
+} else {
+// when there is an offset, the validity slots should be > values
+assert!(
+ceil / 8 <= is_valid.len(),
+"value slots greater than validity slots"
+);
+}
+let mut offset = offset;
+let mut bools = Vec::with_capacity(values.len() / 8 + 1);
+is_valid.iter().for_each(|v| {
+if offset >= 8 {
+offset -= 8;
+return;
+}
+let bin = format!("{:b}", v);
+// TODO: reversing to correct the endianness, find a better 
solution
+// if we have `11001010` it means first value is null, and 8th 
value is valid
+// however if the offset = 3, we should have `11001`, so we skip 
offset
+bin.as_str().chars().rev().skip(offset).for_each(|c| {
+bools.push(c == '1');
+});
+// offset should be covered by available bytes
+offset = 0;
+});
+self.bitmap_builder.append_slice([0..values.len()])?;

Review comment:
   it might be faster to manually call 
`bimap_builder.reserve(values.len())` ahead of time, then call 
`bitmap_builder.append()` in the loop to avoid having to create bools vector. 
`append_slice` is just `append` running in a loop, so if you are already 
looping manually before calling it, then I feel like using `append` would be 
better.

##
File path: rust/arrow/src/array/builder.rs
##
@@ -500,6 +500,26 @@ impl PrimitiveBuilder {
 Ok(())
 }
 
+/// Appends values from a slice of type `T` and a validity byte slice
+pub fn append_values( self, values: &[T::Native], is_valid: &[u8]) -> 
Result<()> {
+let ceil = bit_util::round_upto_power_of_2(values.len(), 8);
+assert_eq!(
+ceil / 8,
+is_valid.len(),
+"value slice not aligned with validity slice"
+);
+let mut bools = Vec::with_capacity(values.len() / 8 + 1);
+is_valid.iter().for_each(|v| {
+let bin = format!("{:b}", v);
+// TODO: reversing to correct the endianness, find a better 
solution
+bin.as_str().chars().rev().for_each(|c| {
+bools.push(c == '1');

Review comment:
   `{:b}` ignores leading zeros, which might not be want we want here. How 
about using bitmaks?
   
   ```rust
   fn main() {
   // 0x38: 0011 1000
   // 0x1F: 0001 
   let bitmap = [0x38, 0x1F];
   bitmap.iter().enumerate().for_each(|(idx, byte)| {
   let mut bitmask = 0x01;
   println!("byte {}: {:b}", idx, byte);
   let offset = 0;
   for _ in offset..8 {
   println!("\t{}", (byte & bitmask) != 0);
   bitmask = bitmask << 1;
   }
   });
   }
   ```
   output:
   
   ```
   byte 0: 111000
false
false
false
true
true
true
false
false
   byte 1: 1
true
true
true
true
true
false
false
false
   ```

##
File path: rust/arrow/src/array/builder.rs
##
@@ -500,6 +500,49 @@ impl PrimitiveBuilder {
 Ok(())
 }
 
+/// Appends values from a slice of type `T` and a validity byte slice
+pub fn append_values(
+ self,
+values: &[T::Native],
+is_valid: &[u8],
+offset: usize,
+) -> Result<()> {
+let ceil = bit_util::round_upto_power_of_2(values.len(), 8);
+if offset == 0 {
+// if there is no offset, the value slice should align with 
validity
+assert_eq!(
+ceil / 8,
+is_valid.len(),
+"value slice not aligned with validity slice"
+);

Review comment:
   just my 2 cents, but i feel like assert should be used for internal 
errors. In this case, it's checking for invalid arguments, which is user/caller 
error, so perhaps returning an Err instead of crashing is better.

##
File path: rust/arrow/src/array/builder.rs
##
@@ -500,6 +500,49 @@ impl 

[GitHub] [arrow] nevi-me commented on a change in pull request #7193: ARROW-7924: [Rust] Add sort for float types

2020-05-30 Thread GitBox


nevi-me commented on a change in pull request #7193:
URL: https://github.com/apache/arrow/pull/7193#discussion_r432901011



##
File path: rust/arrow/src/compute/kernels/sort.rs
##
@@ -283,6 +285,30 @@ mod tests {
 None,
 vec![3, 1, 4, 2, 0, 5],
 );
+test_sort_to_indices_primitive_arrays::(
+vec![
+None,
+Some(-0.05),
+Some(2.225),
+Some(-1.01),
+Some(-0.05),
+None,
+],
+None,
+vec![3, 1, 4, 2, 0, 5],
+);
+test_sort_to_indices_primitive_arrays::(
+vec![
+None,
+Some(-0.05),
+Some(2.225),
+Some(-1.01),
+Some(-0.05),
+None,

Review comment:
   PTAL at my changes, I check for NaN values when I perform the range 
partition, so that there are no nulls when values are sorted.





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.

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




[GitHub] [arrow] wesm commented on pull request #6402: ARROW-7831: [Java] do not allocate a new offset buffer if the slice starts at 0 since the relative offset pointer would be unchanged

2020-05-30 Thread GitBox


wesm commented on pull request #6402:
URL: https://github.com/apache/arrow/pull/6402#issuecomment-636402809


   ping @siddharthteotia or @rymurr 



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.

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




[GitHub] [arrow] github-actions[bot] commented on pull request #7306: ARROW-3688: [Rust] Add append_values for primitive builders

2020-05-30 Thread GitBox


github-actions[bot] commented on pull request #7306:
URL: https://github.com/apache/arrow/pull/7306#issuecomment-636402799


   https://issues.apache.org/jira/browse/ARROW-3688



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.

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




[GitHub] [arrow] wesm commented on pull request #6122: ARROW-7040: [C#] - System.Memory Span.CopyTo - Crashes

2020-05-30 Thread GitBox


wesm commented on pull request #6122:
URL: https://github.com/apache/arrow/pull/6122#issuecomment-636402753


   Following up again on this since 10 weeks have passed



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.

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




[GitHub] [arrow] wesm commented on pull request #7240: ARROW-8792: [C++][Python][R][GLib] New Array compute kernels implementation and execution framework

2020-05-30 Thread GitBox


wesm commented on pull request #7240:
URL: https://github.com/apache/arrow/pull/7240#issuecomment-636402690


   I'll leave this PR open until next week sometime to allow more time for 
comments



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.

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




[GitHub] [arrow] wesm commented on pull request #7294: ARROW-8971: [Python] Upgrade pip

2020-05-30 Thread GitBox


wesm commented on pull request #7294:
URL: https://github.com/apache/arrow/pull/7294#issuecomment-636402407


   Closing per @pitrou's comment. 



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.

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




[GitHub] [arrow] wesm closed pull request #7294: ARROW-8971: [Python] Upgrade pip

2020-05-30 Thread GitBox


wesm closed pull request #7294:
URL: https://github.com/apache/arrow/pull/7294


   



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.

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




[GitHub] [arrow] wesm commented on pull request #7300: ARROW-8844: [C++] Transfer bitmap in words

2020-05-30 Thread GitBox


wesm commented on pull request #7300:
URL: https://github.com/apache/arrow/pull/7300#issuecomment-636401125


   Is there a JIRA about fixing the legibility of the benchmark diff? 



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.

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




[GitHub] [arrow] wesm closed pull request #7301: ARROW-8982: [CI] Remove allow_failures for s390x on TravisCI

2020-05-30 Thread GitBox


wesm closed pull request #7301:
URL: https://github.com/apache/arrow/pull/7301


   



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.

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




[GitHub] [arrow] nealrichardson commented on pull request #7287: ARROW-8771: [C++] Add boost/process library to build support

2020-05-30 Thread GitBox


nealrichardson commented on pull request #7287:
URL: https://github.com/apache/arrow/pull/7287#issuecomment-636384923


   Apologies if I'm slow to iterate on this; since this is a test dependency, 
it doesn't seem urgent, and I've got a few other things I'm juggling at the 
moment.



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.

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




[GitHub] [arrow] nealrichardson commented on pull request #7287: ARROW-8771: [C++] Add boost/process library to build support

2020-05-30 Thread GitBox


nealrichardson commented on pull request #7287:
URL: https://github.com/apache/arrow/pull/7287#issuecomment-636384787


   I just tweaked trim-boost.sh, will need to test that this does the right 
thing locally, then I can update bintray and we can try all of the builds.



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.

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




[GitHub] [arrow] nevi-me commented on a change in pull request #7252: ARROW-8906: [Rust] [DataFusion] support schema inference from multiple CSV files

2020-05-30 Thread GitBox


nevi-me commented on a change in pull request #7252:
URL: https://github.com/apache/arrow/pull/7252#discussion_r432878508



##
File path: rust/arrow/src/datatypes.rs
##
@@ -1289,6 +1417,46 @@ impl Schema {
 Self { fields, metadata }
 }
 
+pub fn try_merge(schemas: ) -> Result {

Review comment:
   Thanks, I'll await a review from another commiter, then we can merge





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.

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




[GitHub] [arrow] houqp commented on a change in pull request #7252: ARROW-8906: [Rust] [DataFusion] support schema inference from multiple CSV files

2020-05-30 Thread GitBox


houqp commented on a change in pull request #7252:
URL: https://github.com/apache/arrow/pull/7252#discussion_r432877858



##
File path: rust/arrow/src/datatypes.rs
##
@@ -1289,6 +1417,46 @@ impl Schema {
 Self { fields, metadata }
 }
 
+pub fn try_merge(schemas: ) -> Result {

Review comment:
   ha, good catch, i added a doc for this, but it was mistakenly added to 
Field's try_merge method. I have fixed and also added doc for Field's try_merge 
method as well.





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.

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




[GitHub] [arrow] nevi-me commented on a change in pull request #7252: ARROW-8906: [Rust] [DataFusion] support schema inference from multiple CSV files

2020-05-30 Thread GitBox


nevi-me commented on a change in pull request #7252:
URL: https://github.com/apache/arrow/pull/7252#discussion_r432873310



##
File path: rust/arrow/src/datatypes.rs
##
@@ -1289,6 +1417,46 @@ impl Schema {
 Self { fields, metadata }
 }
 
+pub fn try_merge(schemas: ) -> Result {

Review comment:
   May you please add a doc for this





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.

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




[GitHub] [arrow] github-actions[bot] commented on pull request #7306: [WIP] ARROW-3688: [Rust] Add append_values for primitive builders

2020-05-30 Thread GitBox


github-actions[bot] commented on pull request #7306:
URL: https://github.com/apache/arrow/pull/7306#issuecomment-636357924


   
   
   Thanks for opening a pull request!
   
   Could you open an issue for this pull request on JIRA?
   https://issues.apache.org/jira/browse/ARROW
   
   Then could you also rename pull request title in the following format?
   
   ARROW-${JIRA_ID}: [${COMPONENT}] ${SUMMARY}
   
   See also:
   
 * [Other pull requests](https://github.com/apache/arrow/pulls/)
 * [Contribution Guidelines - How to contribute 
patches](https://arrow.apache.org/docs/developers/contributing.html#how-to-contribute-patches)
   



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.

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




[GitHub] [arrow] nevi-me commented on a change in pull request #7306: [WIP] ARROW-3688: [Rust] Add append_values for primitive builders

2020-05-30 Thread GitBox


nevi-me commented on a change in pull request #7306:
URL: https://github.com/apache/arrow/pull/7306#discussion_r432870268



##
File path: rust/arrow/src/array/builder.rs
##
@@ -500,6 +500,26 @@ impl PrimitiveBuilder {
 Ok(())
 }
 
+/// Appends values from a slice of type `T` and a validity byte slice
+pub fn append_values( self, values: &[T::Native], is_valid: &[u8]) -> 
Result<()> {
+let ceil = bit_util::round_upto_power_of_2(values.len(), 8);

Review comment:
   The idea here is to check that `&[T::Native; 65]` has validity slice of 
`$[u8; 9]` by rounding 65 to 72, then dividing by 8 to get to 9 u8 values. I'll 
add a test that asserts failure of the condition.





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.

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




[GitHub] [arrow] nevi-me commented on a change in pull request #7306: [WIP] ARROW-3688: [Rust] Add append_values for primitive builders

2020-05-30 Thread GitBox


nevi-me commented on a change in pull request #7306:
URL: https://github.com/apache/arrow/pull/7306#discussion_r432870037



##
File path: rust/arrow/src/util/bit_util.rs
##
@@ -43,7 +43,7 @@ pub fn round_upto_multiple_of_64(num: usize) -> usize {
 
 /// Returns the nearest multiple of `factor` that is `>=` than `num`. Here 
`factor` must
 /// be a power of 2.
-fn round_upto_power_of_2(num: usize, factor: usize) -> usize {
+pub(crate) fn round_upto_power_of_2(num: usize, factor: usize) -> usize {

Review comment:
   exposing this so I can round to 8





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.

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




[GitHub] [arrow] nevi-me commented on a change in pull request #7306: [WIP] ARROW-3688: [Rust] Add append_values for primitive builders

2020-05-30 Thread GitBox


nevi-me commented on a change in pull request #7306:
URL: https://github.com/apache/arrow/pull/7306#discussion_r432869964



##
File path: rust/arrow/src/array/builder.rs
##
@@ -500,6 +500,26 @@ impl PrimitiveBuilder {
 Ok(())
 }
 
+/// Appends values from a slice of type `T` and a validity byte slice
+pub fn append_values( self, values: &[T::Native], is_valid: &[u8]) -> 
Result<()> {
+let ceil = bit_util::round_upto_power_of_2(values.len(), 8);
+assert_eq!(
+ceil / 8,
+is_valid.len(),
+"value slice not aligned with validity slice"
+);
+let mut bools = Vec::with_capacity(values.len() / 8 + 1);
+is_valid.iter().for_each(|v| {
+let bin = format!("{:b}", v);
+// TODO: reversing to correct the endianness, find a better 
solution
+bin.as_str().chars().rev().for_each(|c| {
+bools.push(c == '1');

Review comment:
   Similar to the above comment, I don't know if there's perhaps a better 
way than formatting to binary and iterating through the binary characters.





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.

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




[GitHub] [arrow] nevi-me commented on a change in pull request #7306: [WIP] ARROW-3688: [Rust] Add append_values for primitive builders

2020-05-30 Thread GitBox


nevi-me commented on a change in pull request #7306:
URL: https://github.com/apache/arrow/pull/7306#discussion_r432869911



##
File path: rust/arrow/src/array/builder.rs
##
@@ -500,6 +500,26 @@ impl PrimitiveBuilder {
 Ok(())
 }
 
+/// Appends values from a slice of type `T` and a validity byte slice
+pub fn append_values( self, values: &[T::Native], is_valid: &[u8]) -> 
Result<()> {
+let ceil = bit_util::round_upto_power_of_2(values.len(), 8);
+assert_eq!(
+ceil / 8,
+is_valid.len(),
+"value slice not aligned with validity slice"
+);
+let mut bools = Vec::with_capacity(values.len() / 8 + 1);
+is_valid.iter().for_each(|v| {
+let bin = format!("{:b}", v);
+// TODO: reversing to correct the endianness, find a better 
solution

Review comment:
   I need help here. I'm ultimately trying to convert `&[u8; x]` to 
`&[bool; 8x]` while also handling the correct endianness.
   
   The only way I could find to check endianness is by introducing the 
`byteorder` crate, so I'm unsure if the `rev()` is currently always valid. I'll 
add more tests to check this.





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.

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




[GitHub] [arrow] nevi-me opened a new pull request #7306: [WIP] ARROW-3688: [Rust] Add append_values for primitive builders

2020-05-30 Thread GitBox


nevi-me opened a new pull request #7306:
URL: https://github.com/apache/arrow/pull/7306


   This implements `append_values` for primitive builders, which allows for 
pushing a number of values and their validity buffer at the same time.



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.

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




[GitHub] [arrow] nealrichardson closed pull request #7304: ARROW-8878: [R] try_download is confused when download.file.method isn't default

2020-05-30 Thread GitBox


nealrichardson closed pull request #7304:
URL: https://github.com/apache/arrow/pull/7304


   



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.

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




[GitHub] [arrow] github-actions[bot] commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


github-actions[bot] commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636323393


   Revision: 164aac2e582b84bff6f7c2e06ba3e4a23325aac0
   
   Submitted crossbow builds: [ursa-labs/crossbow @ 
actions-278](https://github.com/ursa-labs/crossbow/branches/all?query=actions-278)
   
   |Task|Status|
   ||--|
   
|conda-clean|[![Azure](https://dev.azure.com/ursa-labs/crossbow/_apis/build/status/ursa-labs.crossbow?branchName=actions-278-azure-conda-clean)](https://dev.azure.com/ursa-labs/crossbow/_build/latest?definitionId=1=actions-278-azure-conda-clean)|



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.

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




[GitHub] [arrow] xhochy commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636323268


   @github-actions crossbow submit conda-clean



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.

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




[GitHub] [arrow] github-actions[bot] commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


github-actions[bot] commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636322844


   Revision: 13ad504f4e692bea0b5bf0bcadecaf79b9ee3b36
   
   Submitted crossbow builds: [ursa-labs/crossbow @ 
actions-277](https://github.com/ursa-labs/crossbow/branches/all?query=actions-277)
   
   |Task|Status|
   ||--|
   
|conda-clean|[![Azure](https://dev.azure.com/ursa-labs/crossbow/_apis/build/status/ursa-labs.crossbow?branchName=actions-277-azure-conda-clean)](https://dev.azure.com/ursa-labs/crossbow/_build/latest?definitionId=1=actions-277-azure-conda-clean)|



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.

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




[GitHub] [arrow] xhochy commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636322673


   @github-actions crossbow submit conda-clean



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.

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




[GitHub] [arrow] github-actions[bot] commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


github-actions[bot] commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636321263


   Revision: c1b8b4139c0d735348916c016ae12e78e3eb3a5b
   
   Submitted crossbow builds: [ursa-labs/crossbow @ 
actions-276](https://github.com/ursa-labs/crossbow/branches/all?query=actions-276)
   
   |Task|Status|
   ||--|
   
|conda-clean|[![Azure](https://dev.azure.com/ursa-labs/crossbow/_apis/build/status/ursa-labs.crossbow?branchName=actions-276-azure-conda-clean)](https://dev.azure.com/ursa-labs/crossbow/_build/latest?definitionId=1=actions-276-azure-conda-clean)|



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.

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




[GitHub] [arrow] xhochy commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636321053


   @github-actions crossbow submit conda-clean



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.

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




[GitHub] [arrow] github-actions[bot] commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


github-actions[bot] commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636320519


   Revision: 246d9cff0c4ff77701a857d5e391e33c19d67fcb
   
   Submitted crossbow builds: [ursa-labs/crossbow @ 
actions-275](https://github.com/ursa-labs/crossbow/branches/all?query=actions-275)
   
   |Task|Status|
   ||--|
   
|conda-clean|[![Azure](https://dev.azure.com/ursa-labs/crossbow/_apis/build/status/ursa-labs.crossbow?branchName=actions-275-azure-conda-clean)](https://dev.azure.com/ursa-labs/crossbow/_build/latest?definitionId=1=actions-275-azure-conda-clean)|



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.

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




[GitHub] [arrow] xhochy commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636320406


   @github-actions crossbow submit conda-clean



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.

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




[GitHub] [arrow] github-actions[bot] commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


github-actions[bot] commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636317212


   Revision: faaecde7b7abb1fbb3889e8d3ac9e138e822efe7
   
   Submitted crossbow builds: [ursa-labs/crossbow @ 
actions-274](https://github.com/ursa-labs/crossbow/branches/all?query=actions-274)
   
   |Task|Status|
   ||--|
   
|conda-clean|[![Azure](https://dev.azure.com/ursa-labs/crossbow/_apis/build/status/ursa-labs.crossbow?branchName=actions-274-azure-conda-clean)](https://dev.azure.com/ursa-labs/crossbow/_build/latest?definitionId=1=actions-274-azure-conda-clean)|



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.

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




[GitHub] [arrow] xhochy commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636317049


   @github-actions crossbow submit conda-clean



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.

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




[GitHub] [arrow] github-actions[bot] commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


github-actions[bot] commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636312986


   Revision: dd8db760dc3393fd566f01e6060ef2a89d824e92
   
   Submitted crossbow builds: [ursa-labs/crossbow @ 
actions-273](https://github.com/ursa-labs/crossbow/branches/all?query=actions-273)
   
   |Task|Status|
   ||--|
   
|conda-clean|[![Azure](https://dev.azure.com/ursa-labs/crossbow/_apis/build/status/ursa-labs.crossbow?branchName=actions-273-azure-conda-clean)](https://dev.azure.com/ursa-labs/crossbow/_build/latest?definitionId=1=actions-273-azure-conda-clean)|



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.

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




[GitHub] [arrow] xhochy commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636312751


   @github-actions crossbow submit conda-clean



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.

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




[GitHub] [arrow] github-actions[bot] commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


github-actions[bot] commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636312265


   Revision: 5b0b30ec636c8fc9c80dd2b9740e2118337ab4d8
   
   Submitted crossbow builds: [ursa-labs/crossbow @ 
actions-272](https://github.com/ursa-labs/crossbow/branches/all?query=actions-272)
   
   |Task|Status|
   ||--|
   
|conda-clean|[![Azure](https://dev.azure.com/ursa-labs/crossbow/_apis/build/status/ursa-labs.crossbow?branchName=actions-272-azure-conda-clean)](https://dev.azure.com/ursa-labs/crossbow/_build/latest?definitionId=1=actions-272-azure-conda-clean)|



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.

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




[GitHub] [arrow] xhochy commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636312087


   @github-actions crossbow submit conda-clean



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.

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




[GitHub] [arrow] github-actions[bot] commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


github-actions[bot] commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636311788


   Revision: df4d0ace16ec43f696fd3a0ed18fb33c0e45324d
   
   Submitted crossbow builds: [ursa-labs/crossbow @ 
actions-271](https://github.com/ursa-labs/crossbow/branches/all?query=actions-271)
   
   |Task|Status|
   ||--|
   
|conda-clean|[![Azure](https://dev.azure.com/ursa-labs/crossbow/_apis/build/status/ursa-labs.crossbow?branchName=actions-271-azure-conda-clean)](https://dev.azure.com/ursa-labs/crossbow/_build/latest?definitionId=1=actions-271-azure-conda-clean)|



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.

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




[GitHub] [arrow] github-actions[bot] commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


github-actions[bot] commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636311574


   https://issues.apache.org/jira/browse/ARROW-8941



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.

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




[GitHub] [arrow] xhochy removed a comment on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy removed a comment on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636311271







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.

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




[GitHub] [arrow] xhochy commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636311589


   @github-actions crossbow submit conda-clean



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.

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




[GitHub] [arrow] xhochy commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636311271


   @github-actions crossbow submit conda-clean



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.

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




[GitHub] [arrow] xhochy commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636311078


   @github-actions crossbow submit conda-clean



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.

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




[GitHub] [arrow] xhochy removed a comment on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy removed a comment on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636311066


   @github-actions crossbow submit conda-cleaqn



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.

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




[GitHub] [arrow] xhochy commented on pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy commented on pull request #7305:
URL: https://github.com/apache/arrow/pull/7305#issuecomment-636311066


   @github-actions crossbow submit conda-cleaqn



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.

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




[GitHub] [arrow] xhochy opened a new pull request #7305: ARROW-8941: [C++/Python] Add cleanup script for arrow-nightlies conda repository

2020-05-30 Thread GitBox


xhochy opened a new pull request #7305:
URL: https://github.com/apache/arrow/pull/7305


   



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.

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