You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "Kengo Seki (Jira)" <ji...@apache.org> on 2019/09/09 02:56:00 UTC
[jira] [Updated] (AVRO-2554) Fix Ruby interop test to read all
supported codecs
[ https://issues.apache.org/jira/browse/AVRO-2554?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kengo Seki updated AVRO-2554:
-----------------------------
Status: Patch Available (was: Open)
> Fix Ruby interop test to read all supported codecs
> --------------------------------------------------
>
> Key: AVRO-2554
> URL: https://issues.apache.org/jira/browse/AVRO-2554
> Project: Apache Avro
> Issue Type: Bug
> Components: interop, ruby
> Reporter: Kengo Seki
> Assignee: Kengo Seki
> Priority: Major
>
> I've just noticed AVRO-2456 had a bug on the Ruby data interop test. It checks only the null codec and doesn't others.
> {code}
> sekikn@4e7deb5781b2:~/avro$ git diff
> diff --git a/lang/ruby/interop/test_interop.rb b/lang/ruby/interop/test_interop.rb
> index a63c158c..f4528f00 100755
> --- a/lang/ruby/interop/test_interop.rb
> +++ b/lang/ruby/interop/test_interop.rb
> @@ -30,6 +30,7 @@ class TestInterop < Test::Unit::TestCase
> sep, codec = File.basename(fn, 'avro').rpartition('_')[1, 2]
> sep.empty? || CODECS_TO_VALIDATE.include?(codec)
> end
> + puts files
>
> files.each do |fn|
> define_method("test_read_#{File.basename(fn, 'avro')}") do
> sekikn@4e7deb5781b2:~/avro$ ls -l build/interop/data
> total 220
> -rw-r--r-- 1 sekikn sekikn 1279 Sep 9 01:53 c.avro
> -rw-r--r-- 1 sekikn sekikn 1258 Sep 9 01:53 csharp.avro
> -rw-r--r-- 1 sekikn sekikn 1240 Sep 9 01:53 csharp_deflate.avro
> -rw-r--r-- 1 sekikn sekikn 38453 Sep 9 01:43 java.avro
> -rw-r--r-- 1 sekikn sekikn 29436 Sep 9 01:43 java_deflate.avro
> -rw-r--r-- 1 sekikn sekikn 35412 Sep 9 01:43 java_snappy.avro
> -rw-r--r-- 1 sekikn sekikn 31929 Sep 9 01:43 java_zstandard.avro
> -rw-r--r-- 1 sekikn sekikn 1204 Sep 9 01:53 perl.avro
> -rw-r--r-- 1 sekikn sekikn 1186 Sep 9 01:53 perl_deflate.avro
> -rw-r--r-- 1 sekikn sekikn 1202 Sep 9 01:53 perl_zstandard.avro
> -rw-r--r-- 1 sekikn sekikn 1138 Sep 9 01:53 php.avro
> -rw-r--r-- 1 sekikn sekikn 1124 Sep 9 01:53 php_deflate.avro
> -rw-r--r-- 1 sekikn sekikn 1390 Sep 9 01:53 py.avro
> -rw-r--r-- 1 sekikn sekikn 1390 Sep 9 01:53 py3.avro
> -rw-r--r-- 1 sekikn sekikn 1375 Sep 9 01:53 py3_deflate.avro
> -rw-r--r-- 1 sekikn sekikn 1384 Sep 9 01:53 py3_snappy.avro
> -rw-r--r-- 1 sekikn sekikn 1388 Sep 9 01:53 py3_zstandard.avro
> -rw-r--r-- 1 sekikn sekikn 1375 Sep 9 01:53 py_deflate.avro
> -rw-r--r-- 1 sekikn sekikn 1384 Sep 9 01:53 py_snappy.avro
> -rw-r--r-- 1 sekikn sekikn 1388 Sep 9 01:53 py_zstandard.avro
> -rw-r--r-- 1 sekikn sekikn 1398 Sep 9 01:53 ruby.avro
> -rw-r--r-- 1 sekikn sekikn 1448 Sep 9 01:53 ruby_deflate.avro
> -rw-r--r-- 1 sekikn sekikn 1411 Sep 9 01:53 ruby_snappy.avro
> -rw-r--r-- 1 sekikn sekikn 1354 Sep 9 01:53 ruby_zstandard.avro
> sekikn@4e7deb5781b2:~/avro$ cd lang/ruby; rake interop
> /home/sekikn/avro/lang/ruby/interop/../../../build/interop/data/ruby.avro
> /home/sekikn/avro/lang/ruby/interop/../../../build/interop/data/java.avro
> /home/sekikn/avro/lang/ruby/interop/../../../build/interop/data/perl.avro
> /home/sekikn/avro/lang/ruby/interop/../../../build/interop/data/py3.avro
> /home/sekikn/avro/lang/ruby/interop/../../../build/interop/data/c.avro
> /home/sekikn/avro/lang/ruby/interop/../../../build/interop/data/py.avro
> /home/sekikn/avro/lang/ruby/interop/../../../build/interop/data/php.avro
> /home/sekikn/avro/lang/ruby/interop/../../../build/interop/data/csharp.avro
> Loaded suite /var/lib/gems/2.3.0/gems/rake-12.3.3/lib/rake/rake_test_loader
> Started
> {code}
> Also, it doesn't test zstd-compressed files for now, though the Ruby bindings supports that codec.
--
This message was sent by Atlassian Jira
(v8.3.2#803003)