apache · jsoref · Nov 21, 2023 · Nov 21, 2023 · Nov 21, 2023 · Nov 21, 2023
@@ -46,7 +46,7 @@ jobs:
           python-version: 3.8
       - name: Install Archery and Crossbow dependencies
         run: pip install -e arrow/dev/archery[bot]
-      - name: Handle Github comment event
+      - name: Handle GitHub comment event
         env:
           ARROW_GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           CROSSBOW_GITHUB_TOKEN: ${{ secrets.CROSSBOW_GITHUB_TOKEN }}

@@ -103,7 +103,7 @@ async function commentNotStartedTicket(github, context, pullRequestNumber) {
 }
 
 /**
- * Assigns the Github Issue to the PR creator.
+ * Assigns the GitHub Issue to the PR creator.
  *
  * @param {Object} github
  * @param {Object} context

@@ -64,7 +64,7 @@ jobs:
               "per_page": 100,
             });
 
-            // this removes non-existent labels
+            // this removes nonexistent labels
             component_labels = component_labels.filter(
               label => repo_labels.data.some(repo_label => repo_label.name === label)
             );

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1459,7 +1459,7 @@
 * [ARROW-12172](https://issues.apache.org/jira/browse/ARROW-12172) - [Python][Packaging] Pass python version as setuptools pretend version in the macOS wheel builds
 * [ARROW-12178](https://issues.apache.org/jira/browse/ARROW-12178) - [CI] Update setuptools in the ubuntu images
 * [ARROW-12186](https://issues.apache.org/jira/browse/ARROW-12186) - [Rust][DataFusion] Fix regexp_match test
-* [ARROW-12209](https://issues.apache.org/jira/browse/ARROW-12209) - [JS] Copy all src files into the the TypeScript package
+* [ARROW-12209](https://issues.apache.org/jira/browse/ARROW-12209) - [JS] Copy all src files into the TypeScript package
 * [ARROW-12220](https://issues.apache.org/jira/browse/ARROW-12220) - [C++][CI] Thread sanitizer failure
 * [ARROW-12226](https://issues.apache.org/jira/browse/ARROW-12226) - [C++] Fix Address Sanitizer failures
 * [ARROW-12227](https://issues.apache.org/jira/browse/ARROW-12227) - [R] Fix RE2 and median nightly build failures
@@ -11430,7 +11430,7 @@
 * [ARROW-67](https://issues.apache.org/jira/browse/ARROW-67) - C++ metadata flatbuffer serialization and data movement to memory maps
 * [ARROW-68](https://issues.apache.org/jira/browse/ARROW-68) - Better error handling for not fully setup systems
 * [ARROW-70](https://issues.apache.org/jira/browse/ARROW-70) - Add adapt 'lite' DCHECK macros from Kudu as also used in Parquet
-* [ARROW-71](https://issues.apache.org/jira/browse/ARROW-71) - [C++] Add clang-tidy and clang-format to the the tool chain.
+* [ARROW-71](https://issues.apache.org/jira/browse/ARROW-71) - [C++] Add clang-tidy and clang-format to the tool chain.
 * [ARROW-73](https://issues.apache.org/jira/browse/ARROW-73) - Support older CMake versions
 * [ARROW-76](https://issues.apache.org/jira/browse/ARROW-76) - Revise format document to include null count, defer non-nullable arrays to the domain of metadata
 * [ARROW-78](https://issues.apache.org/jira/browse/ARROW-78) - C++: Add constructor for DecimalType

@@ -100,7 +100,7 @@ GARROW_AVAILABLE_IN_5_0
 gboolean
 gadataset_file_system_dataset_factory_add_selector(
   GADatasetFileSystemDatasetFactory *factory,
-  GArrorFileSelector *selector,
+  GArrowFileSelector *selector,
   GError **error);
 */
 

@@ -202,7 +202,7 @@ G_DECLARE_DERIVABLE_TYPE(GAFlightSQLServer,
 /**
  * GAFlightSQLServerClass:
  * @get_flight_info_statement: A virtual function to implement
- *   `GetFlightInfoStatment` API that gets a #GAFlightInfo for executing a
+ *   `GetFlightInfoStatement` API that gets a #GAFlightInfo for executing a
  *   SQL query.
  * @do_get_statement: A virtual function to implement `DoGetStatement` API
  *   that gets a #GAFlightDataStream containing the query results.

@@ -5179,7 +5179,7 @@ garrow_binary_dictionary_array_builder_append_indices(GArrowBinaryDictionaryArra
  * garrow_binary_dictionary_array_builder_get_dictionary_length:
  * @builder: A #GArrowBinaryDictionaryArrayBuilder.
  *
- * Returns: A number of entries in the dicitonary.
+ * Returns: A number of entries in the dictionary.
  *
  * Since: 2.0.0
  */
@@ -5413,7 +5413,7 @@ garrow_string_dictionary_array_builder_append_indices(GArrowStringDictionaryArra
  * garrow_string_dictionary_array_builder_get_dictionary_length:
  * @builder: A #GArrowStringDictionaryArrayBuilder.
  *
- * Returns: A number of entries in the dicitonary.
+ * Returns: A number of entries in the dictionary.
  *
  * Since: 2.0.0
  */

@@ -307,7 +307,7 @@ garrow_buffer_get_data(GArrowBuffer *buffer)
  * @buffer: A #GArrowBuffer.
  *
  * Returns: (transfer full) (nullable): The data of the buffer. If the
- *   buffer is imutable, it returns %NULL. The data is owned by the
+ *   buffer is immutable, it returns %NULL. The data is owned by the
  *   buffer. You should not free the data.
  *
  * Since: 0.3.0

@@ -2077,7 +2077,7 @@ garrow_run_end_encoded_array_find_physical_offset(
  *   range of values from offset to length.
  *
  *   Avoid calling this function if the physical length can be
- *   estabilished in some other way (e.g. when iterating over the runs
+ *   established in some other way (e.g. when iterating over the runs
  *   sequentially until the end). This function uses binary-search, so
  *   it has a O(log N) cost.
  *

@@ -230,7 +230,7 @@ G_BEGIN_DECLS
  * #GArrowUTF8NormalizeOptions is a class to customize the
  * `utf8_normalize` function.
  *
- * #GArrowQuantileOptions is a class to customize the `qunatile`
+ * #GArrowQuantileOptions is a class to customize the `quantile`
  * function.
  *
  * #GArrowIndexOptions is a class to customize the `index` function.
@@ -5423,7 +5423,7 @@ garrow_record_batch_take(GArrowRecordBatch *record_batch,
  * @options: (nullable): A #GArrowFilterOptions.
  * @error: (nullable): Return location for a #GError or %NULL.
  *
- * Returns: (nullable) (transfer full): The #GArrowArray filterd
+ * Returns: (nullable) (transfer full): The #GArrowArray filtered
  *   with a boolean selection filter. Nulls in the filter will
  *   result in nulls in the output.
  *
@@ -5659,7 +5659,7 @@ garrow_table_sort_indices(GArrowTable *table,
  * @options: (nullable): A #GArrowFilterOptions.
  * @error: (nullable): Return location for a #GError or %NULL.
  *
- * Returns: (nullable) (transfer full): The #GArrowTable filterd
+ * Returns: (nullable) (transfer full): The #GArrowTable filtered
  *   with a boolean selection filter. Nulls in the filter will
  *   result in nulls in the output.
  *
@@ -5698,7 +5698,7 @@ garrow_table_filter(GArrowTable *table,
  * @options: (nullable): A #GArrowFilterOptions.
  * @error: (nullable): Return location for a #GError or %NULL.
  *
- * Returns: (nullable) (transfer full): The #GArrowTable filterd
+ * Returns: (nullable) (transfer full): The #GArrowTable filtered
  *   with a chunked array filter. Nulls in the filter will
  *   result in nulls in the output.
  *
@@ -5739,7 +5739,7 @@ garrow_table_filter_chunked_array(GArrowTable *table,
  * @options: (nullable): A #GArrowFilterOptions.
  * @error: (nullable): Return location for a #GError or %NULL.
  *
- * Returns: (nullable) (transfer full): The #GArrowChunkedArray filterd
+ * Returns: (nullable) (transfer full): The #GArrowChunkedArray filtered
  *   with a boolean selection filter. Nulls in the filter will
  *   result in nulls in the output.
  *
@@ -5778,7 +5778,7 @@ garrow_chunked_array_filter(GArrowChunkedArray *chunked_array,
  * @options: (nullable): A #GArrowFilterOptions.
  * @error: (nullable): Return location for a #GError or %NULL.
  *
- * Returns: (nullable) (transfer full): The #GArrowChunkedArray filterd
+ * Returns: (nullable) (transfer full): The #GArrowChunkedArray filtered
  *   with a chunked array filter. Nulls in the filter will
  *   result in nulls in the output.
  *
@@ -5819,7 +5819,7 @@ garrow_chunked_array_filter_chunked_array(GArrowChunkedArray *chunked_array,
  * @options: (nullable): A #GArrowFilterOptions.
  * @error: (nullable): Return location for a #GError or %NULL.
  *
- * Returns: (nullable) (transfer full): The #GArrowRecordBatch filterd
+ * Returns: (nullable) (transfer full): The #GArrowRecordBatch filtered
  *   with a boolean selection filter. Nulls in the filter will
  *   result in nulls in the output.
  *

@@ -166,7 +166,7 @@ garrow_field_expression_class_init(GArrowFieldExpressionClass *klass)
  * @reference: A field name or dot path.
  * @error: (nullable): Return location for a #GError or %NULL.
  *
- * Returns: A newly created #GArrowFieldExpression on sucess, %NULL on
+ * Returns: A newly created #GArrowFieldExpression on success, %NULL on
  *   error.
  *
  * Since: 6.0.0

@@ -697,7 +697,7 @@ garrow_file_input_stream_new_file_descriptor(gint file_descriptor,
 
 /**
  * garrow_file_input_stream_get_file_descriptor:
- * @stream: A #GArrowFileInuptStream.
+ * @stream: A #GArrowFileInputStream.
  *
  * Returns: The file descriptor of @stream.
  *

@@ -28,7 +28,7 @@ G_BEGIN_DECLS
  * @title: Local file system classes
  * @include: arrow-glib/arrow-glib.h
  *
- * #GArrowLocalFileSystemOptions is a class for specifyiing options of
+ * #GArrowLocalFileSystemOptions is a class for specifying options of
  * an instance of #GArrowLocalFileSystem.
  *
  * #GArrowLocalFileSystem is a class for an implementation of a file system

@@ -22,7 +22,7 @@
 /**
  * SECTION: metadata-version
  * @title: GArrowMetadataVersion
- * @short_description: Metadata version mapgging between Arrow and arrow-glib
+ * @short_description: Metadata version mapping between Arrow and arrow-glib
  *
  * #GArrowMetadataVersion provides metadata versions corresponding
  * to `arrow::ipc::MetadataVersion` values.

@@ -395,13 +395,13 @@ namespace garrow {
                         int64_t n_bytes) override {
       GError *error = NULL;
       gsize n_written_bytes;
-      auto successed = g_output_stream_write_all(output_stream_,
+      auto succeeded = g_output_stream_write_all(output_stream_,
                                                  data,
                                                  n_bytes,
                                                  &n_written_bytes,
                                                  NULL,
                                                  &error);
-      if (successed) {
+      if (succeeded) {
         position_ += n_written_bytes;
         return arrow::Status::OK();
       } else {
@@ -415,8 +415,8 @@ namespace garrow {
 
     arrow::Status Flush() override {
       GError *error = NULL;
-      auto successed = g_output_stream_flush(output_stream_, NULL, &error);
-      if (successed) {
+      auto succeeded = g_output_stream_flush(output_stream_, NULL, &error);
+      if (succeeded) {
         return arrow::Status::OK();
       } else {
         return garrow_error_to_status(error,

@@ -123,7 +123,7 @@ gparquet_arrow_file_reader_class_init(GParquetArrowFileReaderClass *klass)
 /**
  * gparquet_arrow_file_reader_new_arrow:
  * @source: Arrow source to be read.
- * @error: (nullable): Return locatipcn for a #GError or %NULL.
+ * @error: (nullable): Return location for a #GError or %NULL.
  *
  * Returns: (nullable): A newly created #GParquetArrowFileReader.
  *
@@ -152,7 +152,7 @@ gparquet_arrow_file_reader_new_arrow(GArrowSeekableInputStream *source,
 /**
  * gparquet_arrow_file_reader_new_path:
  * @path: Path to be read.
- * @error: (nullable): Return locatipcn for a #GError or %NULL.
+ * @error: (nullable): Return location for a #GError or %NULL.
  *
  * Returns: (nullable): A newly created #GParquetArrowFileReader.
  *
@@ -189,7 +189,7 @@ gparquet_arrow_file_reader_new_path(const gchar *path,
 /**
  * gparquet_arrow_file_reader_read_table:
  * @reader: A #GParquetArrowFileReader.
- * @error: (nullable): Return locatipcn for a #GError or %NULL.
+ * @error: (nullable): Return location for a #GError or %NULL.
  *
  * Returns: (transfer full) (nullable): A read #GArrowTable.
  *
@@ -220,7 +220,7 @@ gparquet_arrow_file_reader_read_table(GParquetArrowFileReader *reader,
  *   If an index is negative, the index is counted backward from the
  *   end of the columns. `-1` means the last column.
  * @n_column_indices: The number of elements of @column_indices.
- * @error: (nullable): Return locatipcn for a #GError or %NULL.
+ * @error: (nullable): Return location for a #GError or %NULL.
  *
  * Returns: (transfer full) (nullable): A read #GArrowTable.
  *
@@ -273,7 +273,7 @@ gparquet_arrow_file_reader_read_row_group(GParquetArrowFileReader *reader,
 /**
  * gparquet_arrow_file_reader_get_schema:
  * @reader: A #GParquetArrowFileReader.
- * @error: (nullable): Return locatipcn for a #GError or %NULL.
+ * @error: (nullable): Return location for a #GError or %NULL.
  *
  * Returns: (transfer full) (nullable): A got #GArrowSchema.
  *
@@ -302,7 +302,7 @@ gparquet_arrow_file_reader_get_schema(GParquetArrowFileReader *reader,
  * @i: The index of the column to be read.
  *   If an index is negative, the index is counted backward from the
  *   end of the columns. `-1` means the last column.
- * @error: (nullable): Return locatipcn for a #GError or %NULL.
+ * @error: (nullable): Return location for a #GError or %NULL.
  *
  * Returns: (transfer full) (nullable): A read #GArrowChunkedArray.
  *

@@ -406,7 +406,7 @@ gparquet_arrow_file_writer_class_init(GParquetArrowFileWriterClass *klass)
  * @schema: Arrow schema for written data.
  * @sink: Arrow output stream to be written.
  * @writer_properties: (nullable): A #GParquetWriterProperties.
- * @error: (nullable): Return locatipcn for a #GError or %NULL.
+ * @error: (nullable): Return location for a #GError or %NULL.
  *
  * Returns: (nullable): A newly created #GParquetArrowFileWriter.
  *
@@ -451,7 +451,7 @@ gparquet_arrow_file_writer_new_arrow(GArrowSchema *schema,
  * @schema: Arrow schema for written data.
  * @path: Path to be read.
  * @writer_properties: (nullable): A #GParquetWriterProperties.
- * @error: (nullable): Return locatipcn for a #GError or %NULL.
+ * @error: (nullable): Return location for a #GError or %NULL.
  *
  * Returns: (nullable): A newly created #GParquetArrowFileWriter.
  *
@@ -505,7 +505,7 @@ gparquet_arrow_file_writer_new_path(GArrowSchema *schema,
  * @writer: A #GParquetArrowFileWriter.
  * @table: A table to be written.
  * @chunk_size: The max number of rows in a row group.
- * @error: (nullable): Return locatipcn for a #GError or %NULL.
+ * @error: (nullable): Return location for a #GError or %NULL.
  *
  * Returns: %TRUE on success, %FALSE if there was an error.
  *
@@ -528,7 +528,7 @@ gparquet_arrow_file_writer_write_table(GParquetArrowFileWriter *writer,
 /**
  * gparquet_arrow_file_writer_close:
  * @writer: A #GParquetArrowFileWriter.
- * @error: (nullable): Return locatipcn for a #GError or %NULL.
+ * @error: (nullable): Return location for a #GError or %NULL.
  *
  * Returns: %TRUE on success, %FALSE if there was an error.
  *

@@ -169,7 +169,7 @@ gparquet_statistics_get_n_nulls(GParquetStatistics *statistics)
 }
 
 /**
- * gparquet_statistics_has_n_distinct_valuess:
+ * gparquet_statistics_has_n_distinct_values:
  * @statistics: A #GParquetStatistics.
  *
  * Returns: %TRUE if the number of distinct values is set, %FALSE otherwise.

@@ -51,7 +51,7 @@ def test_write_size_limit_bytes
     assert_equal(100, @options.write_size_limit_bytes)
   end
 
-  def test_disable_server_verifiation
+  def test_disable_server_verification
     assert do
       not @options.disable_server_verification?
     end

@@ -59,7 +59,7 @@ def test_to_string
                  modulo.to_s)
   end
 
-  sub_test_case("get_result_nullbale_type") do
+  sub_test_case("get_result_nullable_type") do
     def test_if_null
       assert_equal(Gandiva::ResultNullableType::IF_NULL,
                    @not.result_nullable_type)

@@ -19,7 +19,7 @@ module Helper
   module Readable
     def read_table(input, type: :file)
       if input.is_a?(Arrow::Buffer)
-        input_stream = Arrow::BufferIntputStream.new(input)
+        input_stream = Arrow::BufferInputStream.new(input)
       else
         input_stream = Arrow::FileInputStream.new(input)
       end

@@ -141,12 +141,12 @@ def test_no_diff
 
     def test_diff
       array = build_string_array(["Start", "Shutdown", "Reboot"])
-      other_array = build_string_array(["Start", "Shutdonw", "Reboot"])
+      other_array = build_string_array(["Start", "Shutdown_", "Reboot"])
       assert_equal(<<-STRING.chomp, array.diff_unified(other_array))
 
 @@ -1, +1 @@
 -"Shutdown"
-+"Shutdonw"
++"Shutdown_"
 
       STRING
     end

@@ -41,7 +41,7 @@ def test_scale
     assert_equal(2, data_type.scale)
   end
 
-  def test_deciaml_data_type_new
+  def test_decimal_data_type_new
     assert_equal(Arrow::Decimal128DataType.new(8, 2),
                  Arrow::DecimalDataType.new(8, 2))
   end

@@ -41,7 +41,7 @@ def test_scale
     assert_equal(2, data_type.scale)
   end
 
-  def test_deciaml_data_type_new
+  def test_decimal_data_type_new
     assert_equal(Arrow::Decimal256DataType.new(39, 1),
                  Arrow::DecimalDataType.new(39, 1))
   end

@@ -15,7 +15,7 @@
 # specific language governing permissions and limitations
 # under the License.
 
-class TestDictinaryArrayBuilder < Test::Unit::TestCase
+class TestDictionaryArrayBuilder < Test::Unit::TestCase
   include Helper::Buildable
 
   def setup